Alex Behm has uploaded a new change for review. http://gerrit.cloudera.org:8080/3599
Change subject: IMPALA-3678: Fix migration of predicates into union operands with an order by + limit. ...................................................................... IMPALA-3678: Fix migration of predicates into union operands with an order by + limit. There were two separate issues: First, the SortNode incorrectly picked up unassigned conjuncts, and expected those to be empty. In this case where predicates are migrated into union operands, there could actually be unassigned conjuncts bound by the SortNode's tuple id (and so would be incorrectly picked up). The fix is to not pick up unassigned conjuncts in the SortNode, and allow them to be picked up later (into a SelectNode). Second, when generating the plan for union operands we were missing a call to graft a SelectNode on top of the operand plan to capture unassigned conjuncts. Change-Id: Ie824a1223911ca6f04aa7d99bfe824260dc506fa --- M fe/src/main/java/com/cloudera/impala/planner/SingleNodePlanner.java M fe/src/main/java/com/cloudera/impala/planner/SortNode.java M testdata/workloads/functional-planner/queries/PlannerTest/union.test 3 files changed, 88 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/99/3599/1 -- To view, visit http://gerrit.cloudera.org:8080/3599 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ie824a1223911ca6f04aa7d99bfe824260dc506fa Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Alex Behm <[email protected]>
