Merge branch 'master' into fix-index

vesoft-inc · Nov 23, 2021 · 7692ea6 · 7692ea6
2 parents fc4ed41 + 80e827f
commit 7692ea6
Show file tree

Hide file tree

Showing 39 changed files with 412 additions and 273 deletions.
diff --git a/src/common/expression/test/CMakeLists.txt b/src/common/expression/test/CMakeLists.txt
@@ -34,12 +34,10 @@ set(expression_test_common_libs
     $<TARGET_OBJECTS:version_obj>
     $<TARGET_OBJECTS:graph_session_obj>
     $<TARGET_OBJECTS:graph_flags_obj>
-    $<TARGET_OBJECTS:graph_auth_obj>
     $<TARGET_OBJECTS:util_obj>
     $<TARGET_OBJECTS:expr_visitor_obj>
     $<TARGET_OBJECTS:graph_context_obj>
-    $<TARGET_OBJECTS:planner_obj>
-    $<TARGET_OBJECTS:validator_obj>
+    $<TARGET_OBJECTS:plan_obj>
     $<TARGET_OBJECTS:idgenerator_obj>
     $<TARGET_OBJECTS:graph_obj>
     $<TARGET_OBJECTS:ssl_obj>
@@ -78,16 +76,7 @@ nebula_add_test(
         TypeCastingExpressionTest.cpp
         VersionedVariableExpressionTest.cpp
     OBJECTS
-        $<TARGET_OBJECTS:expression_obj>
-        $<TARGET_OBJECTS:datatypes_obj>
-        $<TARGET_OBJECTS:base_obj>
         $<TARGET_OBJECTS:expr_ctx_mock_obj>
-        $<TARGET_OBJECTS:function_manager_obj>
-        $<TARGET_OBJECTS:wkt_wkb_io_obj>
-        $<TARGET_OBJECTS:agg_function_manager_obj>
-        $<TARGET_OBJECTS:time_obj>
-        $<TARGET_OBJECTS:time_utils_obj>
-        $<TARGET_OBJECTS:fs_obj>
         ${expression_test_common_libs}
     LIBRARIES
         gtest
@@ -101,16 +90,7 @@ nebula_add_executable(
     SOURCES
         ExpressionBenchmark.cpp
     OBJECTS
-        $<TARGET_OBJECTS:expression_obj>
-        $<TARGET_OBJECTS:datatypes_obj>
-        $<TARGET_OBJECTS:base_obj>
         $<TARGET_OBJECTS:expr_ctx_mock_obj>
-        $<TARGET_OBJECTS:function_manager_obj>
-        $<TARGET_OBJECTS:wkt_wkb_io_obj>
-        $<TARGET_OBJECTS:agg_function_manager_obj>
-        $<TARGET_OBJECTS:time_obj>
-        $<TARGET_OBJECTS:time_utils_obj>
-        $<TARGET_OBJECTS:fs_obj>
         ${expression_test_common_libs}
     LIBRARIES
         follybenchmark

diff --git a/src/common/geo/io/wkb/test/CMakeLists.txt b/src/common/geo/io/wkb/test/CMakeLists.txt
@@ -2,53 +2,12 @@
 #
 # This source code is licensed under Apache 2.0 License.
 
-set(WKB_TEST_LIBS
-    $<TARGET_OBJECTS:parser_obj>
-    $<TARGET_OBJECTS:expression_obj>
-    $<TARGET_OBJECTS:encryption_obj>
-    $<TARGET_OBJECTS:network_obj>
-    $<TARGET_OBJECTS:fs_obj>
-    $<TARGET_OBJECTS:stats_obj>
-    $<TARGET_OBJECTS:time_obj>
-    $<TARGET_OBJECTS:common_thrift_obj>
-    $<TARGET_OBJECTS:thrift_obj>
-    $<TARGET_OBJECTS:thread_obj>
-    $<TARGET_OBJECTS:datatypes_obj>
-    $<TARGET_OBJECTS:base_obj>
-    $<TARGET_OBJECTS:function_manager_obj>
-    $<TARGET_OBJECTS:wkt_wkb_io_obj>
-    $<TARGET_OBJECTS:agg_function_manager_obj>
-    $<TARGET_OBJECTS:meta_thrift_obj>
-    $<TARGET_OBJECTS:graph_thrift_obj>
-    $<TARGET_OBJECTS:http_client_obj>
-    $<TARGET_OBJECTS:storage_thrift_obj>
-    $<TARGET_OBJECTS:meta_obj>
-    $<TARGET_OBJECTS:meta_client_obj>
-    $<TARGET_OBJECTS:conf_obj>
-    $<TARGET_OBJECTS:charset_obj>
-    $<TARGET_OBJECTS:file_based_cluster_id_man_obj>
-    $<TARGET_OBJECTS:process_obj>
-    $<TARGET_OBJECTS:time_utils_obj>
-    $<TARGET_OBJECTS:ft_es_graph_adapter_obj>
-    $<TARGET_OBJECTS:ws_common_obj>
-    $<TARGET_OBJECTS:version_obj>
-    $<TARGET_OBJECTS:graph_session_obj>
-    $<TARGET_OBJECTS:graph_flags_obj>
-    $<TARGET_OBJECTS:graph_auth_obj>
-    $<TARGET_OBJECTS:util_obj>
-    $<TARGET_OBJECTS:expr_visitor_obj>
-    $<TARGET_OBJECTS:graph_context_obj>
-    $<TARGET_OBJECTS:planner_obj>
-    $<TARGET_OBJECTS:validator_obj>
-    $<TARGET_OBJECTS:idgenerator_obj>
-    $<TARGET_OBJECTS:graph_obj>
-    $<TARGET_OBJECTS:ssl_obj>
-    $<TARGET_OBJECTS:memory_obj>
-)
-
 nebula_add_test(
     NAME wkb_test
     SOURCES WKBTest.cpp
-    OBJECTS ${WKB_TEST_LIBS}
+    OBJECTS
+      $<TARGET_OBJECTS:datatypes_obj>
+      $<TARGET_OBJECTS:base_obj>
+      $<TARGET_OBJECTS:wkt_wkb_io_obj>
     LIBRARIES gtest gtest_main ${THRIFT_LIBRARIES} ${PROXYGEN_LIBRARIES}
 )
diff --git a/src/common/geo/io/wkt/test/CMakeLists.txt b/src/common/geo/io/wkt/test/CMakeLists.txt
@@ -2,53 +2,12 @@
 #
 # This source code is licensed under Apache 2.0 License.
 
-set(WKT_TEST_LIBS
-    $<TARGET_OBJECTS:parser_obj>
-    $<TARGET_OBJECTS:expression_obj>
-    $<TARGET_OBJECTS:encryption_obj>
-    $<TARGET_OBJECTS:network_obj>
-    $<TARGET_OBJECTS:fs_obj>
-    $<TARGET_OBJECTS:stats_obj>
-    $<TARGET_OBJECTS:time_obj>
-    $<TARGET_OBJECTS:common_thrift_obj>
-    $<TARGET_OBJECTS:thrift_obj>
-    $<TARGET_OBJECTS:thread_obj>
-    $<TARGET_OBJECTS:datatypes_obj>
-    $<TARGET_OBJECTS:base_obj>
-    $<TARGET_OBJECTS:function_manager_obj>
-    $<TARGET_OBJECTS:wkt_wkb_io_obj>
-    $<TARGET_OBJECTS:agg_function_manager_obj>
-    $<TARGET_OBJECTS:meta_thrift_obj>
-    $<TARGET_OBJECTS:graph_thrift_obj>
-    $<TARGET_OBJECTS:http_client_obj>
-    $<TARGET_OBJECTS:storage_thrift_obj>
-    $<TARGET_OBJECTS:meta_obj>
-    $<TARGET_OBJECTS:meta_client_obj>
-    $<TARGET_OBJECTS:conf_obj>
-    $<TARGET_OBJECTS:charset_obj>
-    $<TARGET_OBJECTS:file_based_cluster_id_man_obj>
-    $<TARGET_OBJECTS:process_obj>
-    $<TARGET_OBJECTS:time_utils_obj>
-    $<TARGET_OBJECTS:ft_es_graph_adapter_obj>
-    $<TARGET_OBJECTS:ws_common_obj>
-    $<TARGET_OBJECTS:version_obj>
-    $<TARGET_OBJECTS:graph_session_obj>
-    $<TARGET_OBJECTS:graph_flags_obj>
-    $<TARGET_OBJECTS:graph_auth_obj>
-    $<TARGET_OBJECTS:util_obj>
-    $<TARGET_OBJECTS:expr_visitor_obj>
-    $<TARGET_OBJECTS:graph_context_obj>
-    $<TARGET_OBJECTS:planner_obj>
-    $<TARGET_OBJECTS:validator_obj>
-    $<TARGET_OBJECTS:idgenerator_obj>
-    $<TARGET_OBJECTS:graph_obj>
-    $<TARGET_OBJECTS:ssl_obj>
-    $<TARGET_OBJECTS:memory_obj>
-)
-
 nebula_add_test(
     NAME wkt_test
     SOURCES WKTTest.cpp
-    OBJECTS ${WKT_TEST_LIBS}
+    OBJECTS
+      $<TARGET_OBJECTS:datatypes_obj>
+      $<TARGET_OBJECTS:base_obj>
+      $<TARGET_OBJECTS:wkt_wkb_io_obj>
     LIBRARIES gtest gtest_main ${THRIFT_LIBRARIES} ${PROXYGEN_LIBRARIES}
 )
diff --git a/src/daemons/CMakeLists.txt b/src/daemons/CMakeLists.txt
@@ -118,6 +118,7 @@ nebula_add_executable(
         $<TARGET_OBJECTS:expr_visitor_obj>
         $<TARGET_OBJECTS:optimizer_obj>
         $<TARGET_OBJECTS:planner_obj>
+        $<TARGET_OBJECTS:plan_obj>
         $<TARGET_OBJECTS:executor_obj>
         $<TARGET_OBJECTS:scheduler_obj>
         $<TARGET_OBJECTS:idgenerator_obj>

diff --git a/src/graph/context/test/CMakeLists.txt b/src/graph/context/test/CMakeLists.txt
@@ -34,11 +34,10 @@ SET(CONTEXT_TEST_LIBS
     $<TARGET_OBJECTS:graph_context_obj>
     $<TARGET_OBJECTS:expr_visitor_obj>
     $<TARGET_OBJECTS:parser_obj>
-    $<TARGET_OBJECTS:validator_obj>
     $<TARGET_OBJECTS:graph_flags_obj>
     $<TARGET_OBJECTS:graph_auth_obj>
     $<TARGET_OBJECTS:graph_session_obj>
-    $<TARGET_OBJECTS:planner_obj>
+    $<TARGET_OBJECTS:plan_obj>
     $<TARGET_OBJECTS:idgenerator_obj>
     $<TARGET_OBJECTS:ssl_obj>
     $<TARGET_OBJECTS:memory_obj>

diff --git a/src/graph/executor/test/CMakeLists.txt b/src/graph/executor/test/CMakeLists.txt
@@ -41,6 +41,7 @@ SET(EXEC_QUERY_TEST_OBJS
     $<TARGET_OBJECTS:parser_obj>
     $<TARGET_OBJECTS:validator_obj>
     $<TARGET_OBJECTS:planner_obj>
+    $<TARGET_OBJECTS:plan_obj>
     $<TARGET_OBJECTS:scheduler_obj>
     $<TARGET_OBJECTS:executor_obj>
     $<TARGET_OBJECTS:util_obj>

diff --git a/src/graph/optimizer/test/CMakeLists.txt b/src/graph/optimizer/test/CMakeLists.txt
@@ -39,6 +39,7 @@ set(OPTIMIZER_TEST_LIB
     $<TARGET_OBJECTS:graph_flags_obj>
     $<TARGET_OBJECTS:util_obj>
     $<TARGET_OBJECTS:planner_obj>
+    $<TARGET_OBJECTS:plan_obj>
     $<TARGET_OBJECTS:parser_obj>
     $<TARGET_OBJECTS:graph_context_obj>
     $<TARGET_OBJECTS:validator_obj>

diff --git a/src/graph/planner/CMakeLists.txt b/src/graph/planner/CMakeLists.txt
@@ -26,6 +26,17 @@ nebula_add_library(
     match/PropIndexSeek.cpp
     match/VertexIdSeek.cpp
     match/LabelIndexSeek.cpp
+    ngql/PathPlanner.cpp
+    ngql/GoPlanner.cpp
+    ngql/SubgraphPlanner.cpp
+    ngql/LookupPlanner.cpp
+    ngql/FetchVerticesPlanner.cpp
+    ngql/FetchEdgesPlanner.cpp
+    ngql/MaintainPlanner.cpp
+)
+
+nebula_add_library(
+    plan_obj OBJECT
     plan/PlanNode.cpp
     plan/ExecutionPlan.cpp
     plan/Algo.cpp
@@ -34,11 +45,4 @@ nebula_add_library(
     plan/Query.cpp
     plan/Mutate.cpp
     plan/Maintain.cpp
-    ngql/PathPlanner.cpp
-    ngql/GoPlanner.cpp
-    ngql/SubgraphPlanner.cpp
-    ngql/LookupPlanner.cpp
-    ngql/FetchVerticesPlanner.cpp
-    ngql/FetchEdgesPlanner.cpp
-    ngql/MaintainPlanner.cpp
 )
diff --git a/src/graph/planner/Planner.h b/src/graph/planner/Planner.h
@@ -10,6 +10,7 @@
 
 #include "common/base/Base.h"
 #include "graph/context/ast/AstContext.h"
+#include "graph/planner/plan/ExecutionPlan.h"
 #include "graph/planner/plan/PlanNode.h"
 
 namespace nebula {
@@ -23,12 +24,6 @@ extern const char* kVertexID;
 extern const char* kVertices;
 extern const char* kEdges;
 
-struct SubPlan {
-  // root and tail of a subplan.
-  PlanNode* root{nullptr};
-  PlanNode* tail{nullptr};
-};
-
 std::ostream& operator<<(std::ostream& os, const SubPlan& subplan);
 
 using MatchFunc = std::function<bool(AstContext* astContext)>;

diff --git a/src/graph/planner/plan/ExecutionPlan.h b/src/graph/planner/plan/ExecutionPlan.h
@@ -15,9 +15,14 @@ struct PlanDescription;
 struct PlanNodeDescription;
 
 namespace graph {
-
 class PlanNode;
 
+struct SubPlan {
+  // root and tail of a subplan.
+  PlanNode* root{nullptr};
+  PlanNode* tail{nullptr};
+};
+
 class ExecutionPlan final {
  public:
   explicit ExecutionPlan(PlanNode* root = nullptr);

diff --git a/src/graph/planner/test/CMakeLists.txt b/src/graph/planner/test/CMakeLists.txt
@@ -43,6 +43,7 @@ nebula_add_test(
         $<TARGET_OBJECTS:validator_obj>
         $<TARGET_OBJECTS:expr_visitor_obj>
         $<TARGET_OBJECTS:planner_obj>
+        $<TARGET_OBJECTS:plan_obj>
         $<TARGET_OBJECTS:executor_obj>
         $<TARGET_OBJECTS:scheduler_obj>
         $<TARGET_OBJECTS:util_obj>

diff --git a/src/graph/util/PlannerUtil.cpp b/src/graph/util/PlannerUtil.cpp
@@ -9,7 +9,6 @@
 #include "common/expression/ColumnExpression.h"
 #include "graph/context/QueryContext.h"
 #include "graph/context/ast/QueryAstContext.h"
-#include "graph/planner/Planner.h"
 #include "graph/planner/plan/Query.h"
 
 namespace nebula {

diff --git a/src/graph/util/ValidateUtil.cpp b/src/graph/util/ValidateUtil.cpp
@@ -9,7 +9,6 @@
 #include "common/expression/ColumnExpression.h"
 #include "graph/context/QueryContext.h"
 #include "graph/context/ast/QueryAstContext.h"
-#include "graph/planner/Planner.h"
 #include "graph/planner/plan/Query.h"
 #include "graph/util/ExpressionUtils.h"
 

diff --git a/src/graph/util/test/CMakeLists.txt b/src/graph/util/test/CMakeLists.txt
@@ -39,11 +39,10 @@ nebula_add_test(
         $<TARGET_OBJECTS:graph_auth_obj>
         $<TARGET_OBJECTS:graph_flags_obj>
         $<TARGET_OBJECTS:util_obj>
-        $<TARGET_OBJECTS:planner_obj>
+        $<TARGET_OBJECTS:plan_obj>
         $<TARGET_OBJECTS:parser_obj>
         $<TARGET_OBJECTS:graph_context_obj>
         $<TARGET_OBJECTS:memory_obj>
-        $<TARGET_OBJECTS:validator_obj>
         $<TARGET_OBJECTS:version_obj>
     LIBRARIES
         gtest