address comment

src/graph/validator/MatchValidator.cpp

@@ -187,7 +187,7 @@ Status MatchValidator::buildPathExpr(const MatchPath *path,
   }
   pathBuild->add(InputPropertyExpression::make(pool, nodeInfos.back().alias));
   pathInfo.pathBuild = std::move(pathBuild);
-  pathInfo.anonymous = false;
+  pathInfo.anonymous = pathAlias == nullptr;
   pathInfo.alias = pathAlias == nullptr ? path->toString() : *pathAlias;
   return Status::OK();
 }
@@ -1294,16 +1294,17 @@ Status MatchValidator::validatePathInWhere(
 /*static*/ Status MatchValidator::buildRollUpPathInfo(const MatchPath *path, Path &pathInfo) {
   for (const auto &node : path->nodes()) {
     // The inner variable of expression will be replaced by anno variable
-    if (!node->anonymous()) {
-      pathInfo.compareVariables.emplace_back(node->alias());
+    const auto &nodeAlias = node->alias();
+    if (!nodeAlias.empty() && !AnonVarGenerator::isAnnoVar(nodeAlias)) {
+      pathInfo.compareVariables.emplace_back(nodeAlias);
     }
   }
   for (const auto &edge : path->edges()) {
     const auto &edgeAlias = edge->alias();
-    if (!edge->anonymous()) {
+    if (!edgeAlias.empty() && !AnonVarGenerator::isAnnoVar(edgeAlias)) {
       if (edge->range()) {
         return Status::SemanticError(
-            "Variable '%s` 's type is list. not support used in multiple patterns "
+            "Variable '%s` 's type is edge list. not support used in multiple patterns "
             "simultaneously.",
             edgeAlias.c_str());
       }

src/parser/MatchPath.h

@@ -115,10 +115,6 @@ class MatchEdge final {
     return range_.get();
   }
 
-  bool anonymous() const {
-    return alias_.empty() || alias_.front() == '_';
-  }
-
   std::string toString() const;
 
   MatchEdge clone() const {
@@ -237,10 +233,6 @@ class MatchNode final {
     return alias_;
   }
 
-  bool anonymous() const {
-    return alias_.empty() || alias_.front() == '_';
-  }
-
   const auto* labels() const {
     return labels_.get();
   }

tests/tck/features/match/PathExpr.feature

@@ -448,29 +448,29 @@ Feature: Basic match
       MATCH (v:player{name: 'Tim Duncan'})-[e:like*2]->(n)
        RETURN ()-[e:like*2]->(n)
       """
-    Then a SemanticError should be raised at runtime: Variable 'e` 's type is list. not support used in multiple patterns simultaneously.
+    Then a SemanticError should be raised at runtime: Variable 'e` 's type is edge list. not support used in multiple patterns simultaneously.
     When executing query:
       """
       MATCH (v:player{name: 'Tim Duncan'})-[e:like*3]->(n), (t:team {name: "Spurs"})
       WITH v, e, collect(distinct n) AS ns
         UNWIND [n in ns | ()-[e*3]->(n:player)] AS p
         RETURN p
       """
-    Then a SemanticError should be raised at runtime: Variable 'e` 's type is list. not support used in multiple patterns simultaneously.
+    Then a SemanticError should be raised at runtime: Variable 'e` 's type is edge list. not support used in multiple patterns simultaneously.
     When executing query:
       """
       MATCH (v:player)-[e:like*3]->(n)
         WHERE (n)-[e*3]->(:player)
         RETURN v
       """
-    Then a SemanticError should be raised at runtime: Variable 'e` 's type is list. not support used in multiple patterns simultaneously.
+    Then a SemanticError should be raised at runtime: Variable 'e` 's type is edge list. not support used in multiple patterns simultaneously.
     When executing query:
       """
       MATCH (v:player)-[e:like*1..3]->(n) WHERE (n)-[e*1..4]->(:player) return v
       """
-    Then a SemanticError should be raised at runtime: Variable 'e` 's type is list. not support used in multiple patterns simultaneously.
+    Then a SemanticError should be raised at runtime: Variable 'e` 's type is edge list. not support used in multiple patterns simultaneously.
     When executing query:
       """
       MATCH (v:player)-[e:like*3]->(n) WHERE id(v)=="Tim Duncan" and (n)-[e*3]->(:player) return v
       """
-    Then a SemanticError should be raised at runtime: Variable 'e` 's type is list. not support used in multiple patterns simultaneously.
+    Then a SemanticError should be raised at runtime: Variable 'e` 's type is edge list. not support used in multiple patterns simultaneously.

tests/tck/features/match/PathExprRefLocalVariable.feature

@@ -238,4 +238,4 @@ Feature: Path expression reference local defined variables
       """
       MATCH (v:player{name: 'Tim Duncan'})-[e:like*1..3]->(n), (t:team {name: "Spurs"}) WITH v, e, collect(distinct n) AS ns UNWIND [n in ns | ()-[e*2..4]->(n:player)] AS p RETURN count(p) AS count
       """
-    Then a SemanticError should be raised at runtime: Variable 'e` 's type is list. not support used in multiple patterns simultaneously.
+    Then a SemanticError should be raised at runtime: Variable 'e` 's type is edge list. not support used in multiple patterns simultaneously.