fix clang compiler error

vesoft-inc · Nov 22, 2021 · e9f9830 · e9f9830
1 parent eb10dce
commit e9f9830
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git 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
@@ -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");
   }
 }