fix argument (#4932)

vesoft-inc · Nov 25, 2022 · 758cf61 · 758cf61
1 parent 8842efb
commit 758cf61
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/graph/executor/logic/ArgumentExecutor.cpp b/src/graph/executor/logic/ArgumentExecutor.cpp
@@ -25,7 +25,9 @@ folly::Future<Status> ArgumentExecutor::execute() {
   for (; iter->valid(); iter->next()) {
     auto val = iter->getColumn(alias);
     if (!val.isVertex()) {
-      continue;
+      return Status::Error("Argument only support vertex, but got %s, which is type %s, ",
+                           val.toString().c_str(),
+                           val.typeName().c_str());
     }
     if (unique.emplace(val.getVertex().vid).second) {
       Row row;

diff --git a/src/graph/planner/match/MatchPathPlanner.cpp b/src/graph/planner/match/MatchPathPlanner.cpp
@@ -118,9 +118,9 @@ Status MatchPathPlanner::findStarts(
   allNodeAliasesAvailable.merge(nodeAliasesSeen);
   std::for_each(
       aliasesAvailable.begin(), aliasesAvailable.end(), [&allNodeAliasesAvailable](auto& kv) {
-        if (kv.second == AliasType::kNode) {
-          allNodeAliasesAvailable.emplace(kv.first);
-        }
+        // if (kv.second == AliasType::kNode) {
+        allNodeAliasesAvailable.emplace(kv.first);
+        // }
       });
 
   // Find the start plan node