Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21941 )
Change subject: IMPALA-13461: Added rules to make tpcds queries work. ...................................................................... IMPALA-13461: Added rules to make tpcds queries work. Among the rules that were added: The "Minus" RelNode is not handled directly by the physical node translator and is changed into other nodes that are handled. This was added in the ImpalaMinusToDistinctRule The ExtractLiteralAgg rule compensates for the fact that a literal value cannot be used directly in an agg. The CalciteRelNodeConverter handles breaking down a SubQuery RelNode into simpler RelNodes that can be optimized. The pom.xml file was also changed. There is a java bug in java 8 that causes incremental compiles to fail. So we do a full compile for the Calcite planner now. Change-Id: I03a38aaa5c413b9b4d2f4c179de07935b672a031 Reviewed-on: http://gerrit.cloudera.org:8080/21941 Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Joe McDonnell <[email protected]> --- M java/calcite-planner/pom.xml A java/calcite-planner/src/main/java/org/apache/impala/calcite/rules/ExtractLiteralAgg.java A java/calcite-planner/src/main/java/org/apache/impala/calcite/rules/ImpalaMinusToDistinctRule.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteOptimizer.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteRelNodeConverter.java 5 files changed, 400 insertions(+), 8 deletions(-) Approvals: Impala Public Jenkins: Verified Joe McDonnell: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/21941 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I03a38aaa5c413b9b4d2f4c179de07935b672a031 Gerrit-Change-Number: 21941 Gerrit-PatchSet: 6 Gerrit-Owner: Steve Carlin <[email protected]> Gerrit-Reviewer: Aman Sinha <[email protected]> Gerrit-Reviewer: Anonymous Coward (816) Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Steve Carlin <[email protected]>
