support intersect/minus for cypher

Author: jievince

src/common/expression/test/TestBase.h

@@ -60,7 +60,7 @@ class ExpressionTest : public ::testing::Test {
 
  protected:
   void testExpr(const std::string &exprSymbol, Value expected) {
-    std::string query = "RETURN " + exprSymbol;
+    std::string query = "YIELD " + exprSymbol;
     nebula::graph::QueryContext queryCtxt;
     nebula::GQLParser gParser(&queryCtxt);
     auto result = gParser.parse(query);
@@ -86,7 +86,7 @@ class ExpressionTest : public ::testing::Test {
   }
 
   void testToString(const std::string &exprSymbol, const char *expected) {
-    std::string query = "RETURN " + exprSymbol;
+    std::string query = "YIELD " + exprSymbol;
     nebula::graph::QueryContext queryCtxt;
     nebula::GQLParser gParser(&queryCtxt);
     auto result = gParser.parse(query);
@@ -106,7 +106,7 @@ class ExpressionTest : public ::testing::Test {
   }
 
   void testFunction(const char *name, const std::vector<Value> &args, const Value &expected) {
-    std::string query = "RETURN " + std::string(name) + "(";
+    std::string query = "YIELD " + std::string(name) + "(";
     for (const auto &i : args) {
       query += i.toString() + ",";
     }

src/graph/context/ast/CypherAstContext.h

@@ -158,7 +158,7 @@ struct YieldClauseContext final : CypherClauseContextBase {
   std::vector<Path> paths;
 
   bool distinct{false};
-  const YieldColumns* yieldColumns{nullptr};
+  YieldColumns* yieldColumns{nullptr};
 
   bool hasAgg_{false};
   bool needGenProject_{false};

src/graph/validator/MatchValidator.cpp

@@ -1063,13 +1063,17 @@ Status MatchValidator::checkAlias(
 
 // Validate yield columns.
 // Fill outputs of whole sentence.
-Status MatchValidator::buildOutputs(const YieldColumns *yields) {
+Status MatchValidator::buildOutputs(YieldColumns *yields) {
   for (auto *col : yields->columns()) {
     auto colName = col->name();
     auto typeStatus = deduceExprType(col->expr());
     NG_RETURN_IF_ERROR(typeStatus);
     auto type = typeStatus.value();
     outputs_.emplace_back(colName, type);
+
+    auto foldStatus = ExpressionUtils::foldConstantExpr(col->expr());
+    NG_RETURN_IF_ERROR(foldStatus);
+    col->setExpr(foldStatus.value());
   }
   return Status::OK();
 }

src/graph/validator/MatchValidator.h

@@ -87,7 +87,7 @@ class MatchValidator final : public Validator {
   Status checkAlias(const Expression *refExpr,
                     const std::unordered_map<std::string, AliasType> &aliasesAvailable) const;
 
-  Status buildOutputs(const YieldColumns *yields);
+  Status buildOutputs(YieldColumns *yields);
 
   StatusOr<Expression *> makeEdgeSubFilter(MapExpression *map) const;
 

tests/tck/features/expression/Case.feature

@@ -298,7 +298,7 @@ Feature: Case Expression
   Scenario: Using the return value of case expr as an input
     When executing query:
       """
-      RETURN CASE WHEN true THEN "Tim Duncan" ELSE "ABC" END AS a | GO FROM $-.a OVER like YIELD like._dst;
+      YIELD CASE WHEN true THEN "Tim Duncan" ELSE "ABC" END AS a | GO FROM $-.a OVER like YIELD like._dst;
       """
     Then the result should be, in order:
       | like._dst       |

tests/tck/features/expression/function/Mathematical.feature

@@ -3,6 +3,9 @@
 # This source code is licensed under Apache 2.0 License.
 Feature: Mathematical function Expression
 
+  Background:
+    Given a graph with space named "nba"
+
   Scenario: bit functions
     When executing query:
       """

tests/tck/features/function/coalesce.feature

@@ -1,7 +1,10 @@
+# Copyright (c) 2022 vesoft inc. All rights reserved.
+#
+# This source code is licensed under Apache 2.0 License.
 Feature: Coalesce Function
 
   Background:
-    Test coalesce function
+    Given a graph with space named "nba"
 
   Scenario: test normal case
     When executing query:

tests/tck/features/yield/return.feature

@@ -4,7 +4,7 @@
 Feature: Return
 
   Background:
-    Given an empty graph
+    Given a graph with space named "nba"
 
   Scenario: base
     When executing query:
@@ -79,4 +79,4 @@ Feature: Return
       """
       RETURN name
       """
-    Then a SemanticError should be raised at runtime: Invalid label identifiers: name
+    Then a SemanticError should be raised at runtime: Alias used but not defined: `name'