Thomas Tauber-Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9218
Change subject: IMPALA-6473: Fix analytic fn that partitions and orders on same expr ...................................................................... IMPALA-6473: Fix analytic fn that partitions and orders on same expr Previously, an analytic function that used the same expr in both the 'partition by' and 'order by' clauses, and where the expr meets the criteria for being materialized before the sort, would hit an IllegalStateException due to the expr being inserted into the same ExprSubstitutionMap twice. If the values have already been partitioned on the expr, then all of the values for it in each partition will be the same and also ordering on the expr doesn't change the results. So, the fix is to simply exclude the duplicate expr from the 'order by' while still partitioning on it. Testing: - Added a regression test to analytic-fns.test Change-Id: Id5f1d5fbc6f69df5850f96afed345ce27668c30b --- M fe/src/main/java/org/apache/impala/planner/AnalyticPlanner.java M testdata/workloads/functional-query/queries/QueryTest/analytic-fns.test 2 files changed, 24 insertions(+), 3 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/18/9218/1 -- To view, visit http://gerrit.cloudera.org:8080/9218 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Id5f1d5fbc6f69df5850f96afed345ce27668c30b Gerrit-Change-Number: 9218 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Tauber-Marshall <[email protected]>
