diff --git a/src/graph/validator/FindPathValidator.cpp b/src/graph/validator/FindPathValidator.cpp index ffdcfc722fa..8fcf186f2df 100644 --- a/src/graph/validator/FindPathValidator.cpp +++ b/src/graph/validator/FindPathValidator.cpp @@ -64,14 +64,14 @@ Status FindPathValidator::validateWhere(WhereClause* where) { Status FindPathValidator::validateYield(YieldClause* yield) { if (yield == nullptr) { - return Status::SemanticError("missing yield clause."); + return Status::SemanticError("Missing yield clause."); } if (yield->columns().size() != 1) { - return Status::SemanticError("only support yield path"); + return Status::SemanticError("Only support yield path"); } - const auto& col = yield->columns().front(); + auto col = yield->columns().front(); if (col->expr()->kind() != Expression::Kind::kLabel || col->expr()->toString() != "PATH") { - return Status::SemanticError("illegal yield clauses `%s'. only support yield path", + return Status::SemanticError("Illegal yield clauses `%s'. only support yield path", col->toString().c_str()); } outputs_.emplace_back(col->name(), Value::Type::PATH); diff --git a/src/graph/validator/test/FindPathValidatorTest.cpp b/src/graph/validator/test/FindPathValidatorTest.cpp index 8bd30387522..03ba19fc4d5 100644 --- a/src/graph/validator/test/FindPathValidatorTest.cpp +++ b/src/graph/validator/test/FindPathValidatorTest.cpp @@ -21,7 +21,7 @@ TEST_F(FindPathValidatorTest, invalidYield) { { std::string query = "FIND SHORTEST PATH FROM \"Tim\" TO \"Tony\" OVER *"; auto result = checkResult(query); - EXPECT_EQ(std::string(result.message()), "SemanticError: missing yield clause."); + EXPECT_EQ(std::string(result.message()), "SemanticError: Missing yield clause."); } { std::string query = "FIND SHORTEST PATH FROM \"Tim\" TO \"Tony\" OVER * YIELD vertex"; @@ -34,7 +34,7 @@ TEST_F(FindPathValidatorTest, invalidYield) { "FIND ALL PATH WITH PROP FROM \"Tim\" TO \"Tony\" OVER like YIELD edge as e"; auto result = checkResult(query); EXPECT_EQ(std::string(result.message()), - "SemanticError: illegal yield clauses `EDGE AS e'. only support yield path"); + "SemanticError: Illegal yield clauses `EDGE AS e'. only support yield path"); } { std::string query = @@ -49,7 +49,7 @@ TEST_F(FindPathValidatorTest, invalidYield) { "$$.player.name"; auto result = checkResult(query); EXPECT_EQ(std::string(result.message()), - "SemanticError: illegal yield clauses `$$.player.name'. only support yield path"); + "SemanticError: Illegal yield clauses `$$.player.name'. only support yield path"); } }