Skip to content

Commit

Permalink
fix argument (#4932)
Browse files Browse the repository at this point in the history
  • Loading branch information
jievince authored Nov 25, 2022
1 parent 8842efb commit 758cf61
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
4 changes: 3 additions & 1 deletion src/graph/executor/logic/ArgumentExecutor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down
6 changes: 3 additions & 3 deletions src/graph/planner/match/MatchPathPlanner.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 758cf61

Please sign in to comment.