Skip to content

Commit c8e5923

Browse files
committed
fix: left & right in ComparisonExpression should be final
1 parent 8c558c4 commit c8e5923

File tree

2 files changed

+2
-12
lines changed

2 files changed

+2
-12
lines changed

iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/relational/planner/ir/PredicateWithUncorrelatedScalarSubqueryReconstructor.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -93,8 +93,6 @@ public void reconstructPredicateWithUncorrelatedScalarSubquery(
9393
context, (SubqueryExpression) left, analysis.getWith());
9494
result.ifPresent(comparisonExpression::setShadowLeft);
9595
}
96-
comparisonExpression.setLeft(left);
97-
comparisonExpression.setRight(right);
9896
}
9997
}
10098

iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/relational/sql/ast/ComparisonExpression.java

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -96,9 +96,9 @@ public Operator negate() {
9696
}
9797

9898
private final Operator operator;
99-
private Expression left;
99+
private final Expression left;
100100
private Expression shadowLeft;
101-
private Expression right;
101+
private final Expression right;
102102
private Expression shadowRight;
103103

104104
public ComparisonExpression(Operator operator, Expression left, Expression right) {
@@ -136,14 +136,6 @@ public Expression getRight() {
136136
return shadowRight != null ? shadowRight : right;
137137
}
138138

139-
public void setLeft(Expression left) {
140-
this.left = left;
141-
}
142-
143-
public void setRight(Expression right) {
144-
this.right = right;
145-
}
146-
147139
@Override
148140
public <R, C> R accept(AstVisitor<R, C> visitor, C context) {
149141
return visitor.visitComparisonExpression(this, context);

0 commit comments

Comments
 (0)