Sahil Takiar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14144
Change subject: IMPALA-8890: Advance read page in UnpinStream ...................................................................... IMPALA-8890: Advance read page in UnpinStream Fixes a DCHECK in BufferedTupleStream::UnpinStream where ExpectedPinCount would fail if the stream still referenced a read page that had be completely read. The patch changes UnpinStream so that if the current read page has been completely read, the read page is advanced using NextReadPage(). Testing: * Ran core tests * Added new tests to buffered-tuple-stream-test Change-Id: Id61a82559fd2baab237cc44f60a69cba2b30c16e --- M be/src/exec/analytic-eval-node.cc M be/src/exec/grouping-aggregator-partition.cc M be/src/exec/grouping-aggregator.cc M be/src/exec/grouping-aggregator.h M be/src/exec/partitioned-hash-join-node.cc M be/src/runtime/buffered-tuple-stream-test.cc M be/src/runtime/buffered-tuple-stream.cc M be/src/runtime/buffered-tuple-stream.h M be/src/runtime/spillable-row-batch-queue.cc 9 files changed, 106 insertions(+), 24 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/44/14144/1 -- To view, visit http://gerrit.cloudera.org:8080/14144 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Id61a82559fd2baab237cc44f60a69cba2b30c16e Gerrit-Change-Number: 14144 Gerrit-PatchSet: 1 Gerrit-Owner: Sahil Takiar <[email protected]>
