Skip to content

Commit 9148512

Browse files
committed
remove useless code
Signed-off-by: xinyual <xinyual@amazon.com>
1 parent 93eb5b3 commit 9148512

1 file changed

Lines changed: 0 additions & 51 deletions

File tree

core/src/main/java/org/opensearch/sql/calcite/CalcitePlanContext.java

Lines changed: 0 additions & 51 deletions
Original file line numberDiff line numberDiff line change
@@ -16,14 +16,11 @@
1616
import java.util.function.BiFunction;
1717
import lombok.Getter;
1818
import lombok.Setter;
19-
import org.apache.calcite.plan.RelOptCluster;
20-
import org.apache.calcite.plan.RelOptSchema;
2119
import org.apache.calcite.rex.RexCorrelVariable;
2220
import org.apache.calcite.rex.RexLambdaRef;
2321
import org.apache.calcite.rex.RexNode;
2422
import org.apache.calcite.tools.FrameworkConfig;
2523
import org.apache.calcite.tools.RelBuilder;
26-
import org.apache.calcite.tools.RelBuilderFactory;
2724
import org.opensearch.sql.ast.expression.UnresolvedExpression;
2825
import org.opensearch.sql.calcite.utils.CalciteToolsHelper;
2926
import org.opensearch.sql.common.setting.Settings;
@@ -75,25 +72,6 @@ private CalcitePlanContext(FrameworkConfig config, SysLimit sysLimit, QueryType
7572
this.rexLambdaRefMap = new HashMap<>();
7673
}
7774

78-
private CalcitePlanContext(
79-
FrameworkConfig config,
80-
Integer querySizeLimit,
81-
QueryType queryType,
82-
Connection connection,
83-
RelBuilder relBuilder,
84-
ExtendedRexBuilder rexBuilder,
85-
FunctionProperties functionProperties,
86-
Map<String, RexLambdaRef> rexLambdaRefMap) {
87-
this.config = config;
88-
this.querySizeLimit = querySizeLimit;
89-
this.queryType = queryType;
90-
this.connection = connection;
91-
this.relBuilder = relBuilder;
92-
this.rexBuilder = rexBuilder;
93-
this.functionProperties = functionProperties;
94-
this.rexLambdaRefMap = rexLambdaRefMap;
95-
}
96-
9775
public RexNode resolveJoinCondition(
9876
UnresolvedExpression expr,
9977
BiFunction<UnresolvedExpression, CalcitePlanContext, RexNode> transformFunction) {
@@ -127,35 +105,6 @@ public CalcitePlanContext clone() {
127105
return new CalcitePlanContext(config, sysLimit, queryType);
128106
}
129107

130-
/**
131-
* A deep copy to create a totally same one calciteContext
132-
*
133-
* @return a deep clone calcite context and current context
134-
*/
135-
public CalcitePlanContext deepClone() {
136-
RelOptCluster cluster = this.relBuilder.getCluster();
137-
RelBuilderFactory factory = RelBuilder.proto(config.getContext());
138-
RelOptSchema schema =
139-
this.relBuilder.getCluster().getPlanner().getContext().unwrap(RelOptSchema.class);
140-
RelBuilder siblingRelBuilder = factory.create(cluster, schema);
141-
siblingRelBuilder.push(this.relBuilder.peek()); // Add current logical plan as base
142-
CalcitePlanContext clone =
143-
new CalcitePlanContext(
144-
config,
145-
querySizeLimit,
146-
queryType,
147-
connection,
148-
siblingRelBuilder,
149-
new ExtendedRexBuilder(siblingRelBuilder.getRexBuilder()),
150-
functionProperties,
151-
rexLambdaRefMap);
152-
clone.inCoalesceFunction = this.inCoalesceFunction;
153-
clone.isProjectVisited = this.isProjectVisited;
154-
clone.isResolvingJoinCondition = this.isResolvingJoinCondition;
155-
clone.isResolvingSubquery = this.isResolvingSubquery;
156-
return clone;
157-
}
158-
159108
public static CalcitePlanContext create(
160109
FrameworkConfig config, SysLimit sysLimit, QueryType queryType) {
161110
return new CalcitePlanContext(config, sysLimit, queryType);

0 commit comments

Comments
 (0)