Michael Smith has uploaded this change for review. ( http://gerrit.cloudera.org:8080/21679
Change subject: IMPALA-13302: Analyze new rewrite exprs ...................................................................... IMPALA-13302: Analyze new rewrite exprs Ensures all new expressions created during ExprRewriteRule evaluation are analyzed so later rules will evaluate them. Centralize analyzing and logging new expressions created by rewrite rules. Fixes reAnalyze marking conjunct assignment before new conjuncts are registered. Adds ExprRewriteRules tests that previously hit the new Precondition in markConjunctAssigned, and rewrite PlannerTest. Change-Id: I6be731c2ea79c96e51d199c822e2cb34e5bb3028 --- M fe/src/main/java/org/apache/impala/analysis/Analyzer.java M fe/src/main/java/org/apache/impala/rewrite/BetweenToCompoundRule.java M fe/src/main/java/org/apache/impala/rewrite/CountDistinctToNdvRule.java M fe/src/main/java/org/apache/impala/rewrite/DefaultNdvScaleRule.java M fe/src/main/java/org/apache/impala/rewrite/ExprRewriter.java M fe/src/main/java/org/apache/impala/rewrite/ExtractCommonConjunctRule.java M fe/src/main/java/org/apache/impala/rewrite/NormalizeExprsRule.java M fe/src/main/java/org/apache/impala/rewrite/SimplifyCastStringToTimestamp.java M fe/src/test/java/org/apache/impala/analysis/ExprRewriteRulesTest.java M fe/src/test/java/org/apache/impala/planner/PlannerTest.java A testdata/workloads/functional-planner/queries/PlannerTest/rewrite.test 11 files changed, 153 insertions(+), 37 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/79/21679/1 -- To view, visit http://gerrit.cloudera.org:8080/21679 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I6be731c2ea79c96e51d199c822e2cb34e5bb3028 Gerrit-Change-Number: 21679 Gerrit-PatchSet: 1 Gerrit-Owner: Michael Smith <[email protected]>
