Steve Carlin has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/22117 )
Change subject: IMPALA-13582: Calcite planner: return proper labels for columns ...................................................................... IMPALA-13582: Calcite planner: return proper labels for columns The field names were not getting passed up to the output expressions. These are found on the RelNode row type object. The change is made in two different flows: The first flow is in CalciteSingleNodePlanner which gets hit when running from impala-shell and the use_calcite_planner query option is used. The second flow is in ExecRequestCreator and gets hit when running with the start-up option that loads a different JniFrontend jar. This mode will soon be deprecated, but is still used for testing purposes. Change-Id: I42818646d98f87d8744585010fc166f9d416aec1 --- M java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaAggRel.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaAnalyticRel.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaHdfsScanRel.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaJoinRel.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaProjectRel.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaSortRel.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaValuesRel.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/NodeCreationUtils.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/NodeWithExprs.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteSingleNodePlanner.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/service/ExecRequestCreator.java 11 files changed, 29 insertions(+), 34 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/17/22117/3 -- To view, visit http://gerrit.cloudera.org:8080/22117 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I42818646d98f87d8744585010fc166f9d416aec1 Gerrit-Change-Number: 22117 Gerrit-PatchSet: 3 Gerrit-Owner: Steve Carlin <[email protected]>
