enhance/appendVerticeExecutor

vesoft-inc · Dec 7, 2022 · 0bc1dd9 · 0bc1dd9
1 parent 3762db7
commit 0bc1dd9
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/src/graph/executor/query/AppendVerticesExecutor.cpp b/src/graph/executor/query/AppendVerticesExecutor.cpp
@@ -27,7 +27,7 @@ StatusOr<DataSet> AppendVerticesExecutor::buildRequestDataSet(const AppendVertic
 folly::Future<Status> AppendVerticesExecutor::appendVertices() {
   SCOPED_TIMER(&execTime_);
   auto *av = asNode<AppendVertices>(node());
-  if (FLAGS_optimize_appendvertices && av != nullptr && av->props() == nullptr) {
+  if (FLAGS_optimize_appendvertices && av != nullptr && av->noNeedFetchProp()) {
     return handleNullProp(av);
   }
 

diff --git a/src/graph/planner/plan/Query.h b/src/graph/planner/plan/Query.h
@@ -346,6 +346,19 @@ class GetVertices : public Explore {
     return props_.get();
   }
 
+  bool noNeedFetchProp() const {
+    if (props_.get() == nullptr) {
+      return true;
+    }
+    auto& props = *props_;
+    for (const auto& prop : props) {
+      if (!prop.get_props().empty()) {
+        return false;
+      }
+    }
+    return true;
+  }
+
   const std::vector<Expr>* exprs() const {
     return exprs_.get();
   }