Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12562 )
Change subject: IMPALA-8069: Crash in impala::Sorter::Run::Run ...................................................................... IMPALA-8069: Crash in impala::Sorter::Run::Run Modifies the planner to omit a sorter node in the obscure case we have an analytic query with a constant sort. The analytic node requires a buffered tuple. When the sort is omitted, creates a buffered tuple on top of the node that would have been input to the sort. Testing: * Added a PlannerTest case. * Added an end-to-end test. * Reran the query that previously crashed: it now produces proper results. Change-Id: I19a57f3791ce9e41fe0ba79dc5834e762341c74e Reviewed-on: http://gerrit.cloudera.org:8080/12562 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/planner/AnalyticPlanner.java M testdata/workloads/functional-planner/queries/PlannerTest/analytic-fns.test M testdata/workloads/functional-planner/queries/PlannerTest/order.test M testdata/workloads/functional-query/queries/QueryTest/analytic-fns.test 4 files changed, 71 insertions(+), 17 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/12562 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I19a57f3791ce9e41fe0ba79dc5834e762341c74e Gerrit-Change-Number: 12562 Gerrit-PatchSet: 7 Gerrit-Owner: Paul Rogers <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Paul Rogers <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
