Skip to content

Commit 5564f26

Browse files
committed
Trim unused fields after when converting sql back to rel (1857/2027)
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
1 parent 6cf5b21 commit 5564f26

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

core/src/main/java/org/opensearch/sql/executor/QueryService.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@
2727
import org.apache.calcite.rel.RelCollation;
2828
import org.apache.calcite.rel.RelCollations;
2929
import org.apache.calcite.rel.RelNode;
30-
import org.apache.calcite.rel.RelRoot;
3130
import org.apache.calcite.rel.core.Sort;
3231
import org.apache.calcite.rel.logical.LogicalSort;
3332
import org.apache.calcite.rel.rel2sql.RelToSqlConverter;
@@ -399,8 +398,8 @@ public SqlNode visit(SqlIdentifier id) {
399398
cluster,
400399
PplConvertletTable.INSTANCE,
401400
sql2relConfig);
402-
RelRoot validatedRelRoot = sql2rel.convertQuery(rewritten, false, true);
403-
return validatedRelRoot.rel.accept(new PplRelToSqlRelShuttle(context.rexBuilder, false));
401+
RelNode validatedRel = sql2rel.convertQuery(rewritten, false, true).project();
402+
return validatedRel.accept(new PplRelToSqlRelShuttle(context.rexBuilder, false));
404403
}
405404

406405
/** Translate {@link LogicalPlan} to {@link PhysicalPlan}. */

0 commit comments

Comments
 (0)