Skip to content

Commit

Permalink
fix conflict
Browse files Browse the repository at this point in the history
  • Loading branch information
nevermore3 committed Aug 18, 2022
1 parent 16ff6e1 commit 7a98297
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 deletions.
7 changes: 3 additions & 4 deletions src/graph/visitor/PropertyTrackerVisitor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -224,10 +224,9 @@ void PropertyTrackerVisitor::visit(AggregateExpression *expr) {
auto funName = expr->name();
std::transform(funName.begin(), funName.end(), funName.begin(), ::tolower);
if (funName == "count") {
auto *argExpr = expr->arg();
if (argExpr->kind() == Expression::Kind::kConstant ||
argExpr->kind() == Expression::Kind::kInputProperty ||
argExpr->kind() == Expression::Kind::kVarProperty) {
auto kind = expr->arg()->kind();
if (kind == Expression::Kind::kConstant || kind == Expression::Kind::kInputProperty ||
kind == Expression::Kind::kVarProperty) {
return;
}
}
Expand Down
1 change: 0 additions & 1 deletion tests/tck/features/optimizer/PrunePropertiesRule.feature
Original file line number Diff line number Diff line change
Expand Up @@ -499,7 +499,6 @@ Feature: Prune Properties rule
| "Steve Nash" |
And the execution plan should be:
| id | name | dependencies | operator info |
| 15 | DataCollect | 14 | |
| 14 | Dedup | 13 | |
| 13 | Union | 18, 19 | |
| 18 | Project | 4 | |
Expand Down

0 comments on commit 7a98297

Please sign in to comment.