Skip to content

Commit

Permalink
Address comments.
Browse files Browse the repository at this point in the history
  • Loading branch information
CPWstatic committed Mar 7, 2022
1 parent 2906f6a commit f8bf624
Show file tree
Hide file tree
Showing 8 changed files with 8 additions and 10 deletions.
2 changes: 1 addition & 1 deletion src/graph/planner/match/ArgumentFinder.h
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@

namespace nebula {
namespace graph {
// ArgumentFinder find if a pattern use a named alias that already declared
// ArgumentFinder finds if a pattern use a named alias that already declared
// in former patterns.
class ArgumentFinder final : public StartVidFinder {
public:
Expand Down
3 changes: 1 addition & 2 deletions src/graph/planner/match/LabelIndexSeek.h
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,7 @@
namespace nebula {
namespace graph {

// The LabelIndexSeek was designed to find if could get the starting vids by tag
// index.
// The LabelIndexSeek finds if a plan could get the starting vids by tag index.
class LabelIndexSeek final : public StartVidFinder {
public:
static std::unique_ptr<LabelIndexSeek> make() {
Expand Down
2 changes: 1 addition & 1 deletion src/graph/planner/match/MatchPlanner.h
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@

namespace nebula {
namespace graph {
// MatchPlanner generate plans for Match statement based on AstContext.
// MatchPlanner generates plans for Match statement based on AstContext.
class MatchPlanner final : public Planner {
public:
static std::unique_ptr<MatchPlanner> make() {
Expand Down
2 changes: 1 addition & 1 deletion src/graph/planner/match/OrderByClausePlanner.h
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@

namespace nebula {
namespace graph {
// The OrderByClausePlanner generate plan for order by clause;
// The OrderByClausePlanner generates plan for order by clause;
class OrderByClausePlanner final : public CypherClausePlanner {
public:
OrderByClausePlanner() = default;
Expand Down
2 changes: 1 addition & 1 deletion src/graph/planner/match/PropIndexSeek.h
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@

namespace nebula {
namespace graph {
// The PropIndexSeek find if a plan could get starting vids by tag
// The PropIndexSeek finds if a plan could get starting vids by tag
// props or edge props index.
class PropIndexSeek final : public StartVidFinder {
public:
Expand Down
2 changes: 1 addition & 1 deletion src/graph/planner/match/ReturnClausePlanner.h
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@

namespace nebula {
namespace graph {
// The ReturnClausePlanner generate plan for return clause.
// The ReturnClausePlanner generates plan for return clause.
class ReturnClausePlanner final : public CypherClausePlanner {
public:
ReturnClausePlanner() = default;
Expand Down
3 changes: 1 addition & 2 deletions src/graph/planner/match/ScanSeek.h
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,7 @@

namespace nebula {
namespace graph {
// The ScanSeek find if a plan could get the starting vids in
// filter.
// The ScanSeek finds if a plan could get the starting vids in filter.
class ScanSeek final : public StartVidFinder {
public:
static std::unique_ptr<ScanSeek> make() {
Expand Down
2 changes: 1 addition & 1 deletion src/graph/planner/plan/Query.h
Original file line number Diff line number Diff line change
Expand Up @@ -1611,7 +1611,7 @@ class BiLeftJoin final : public BiJoin {
void cloneMembers(const BiLeftJoin&);
};

// Lnner join
// Inner join
class BiInnerJoin final : public BiJoin {
public:
static BiInnerJoin* make(QueryContext* qctx,
Expand Down

0 comments on commit f8bf624

Please sign in to comment.