Steve Carlin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23691 )
Change subject: IMPALA-14434: Calcite planner: implement partition key scan optimization ...................................................................... IMPALA-14434: Calcite planner: implement partition key scan optimization Implemented the partition key scan optimization for Calcite planner Most of the code was already in place. Just needed to refactor some code in SingleNodePlanner to make it callable from Calcite and use the already created isPartitionKeyScan method in ImpalaHdfsScanRel. Testing was done by running the Impala e2e tests with the use_calcite_planner flag set to true. Change-Id: I7b5b8a8115f65f6be27a5be0e19f21eebab61a32 Reviewed-on: http://gerrit.cloudera.org:8080/23691 Reviewed-by: Joe McDonnell <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Steve Carlin <[email protected]> --- M fe/src/main/java/org/apache/impala/planner/SingleNodePlanner.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaHdfsScanRel.java M testdata/workloads/functional-query/queries/QueryTest/calcite.test 3 files changed, 65 insertions(+), 30 deletions(-) Approvals: Joe McDonnell: Looks good to me, but someone else must approve Impala Public Jenkins: Verified Steve Carlin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/23691 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I7b5b8a8115f65f6be27a5be0e19f21eebab61a32 Gerrit-Change-Number: 23691 Gerrit-PatchSet: 3 Gerrit-Owner: Steve Carlin <[email protected]> Gerrit-Reviewer: Aman Sinha <[email protected]> Gerrit-Reviewer: Fang-Yu Rao <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Pranav Lodha <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Reviewer: Steve Carlin <[email protected]>
