Tim Armstrong has uploaded a new patch set (#3). Change subject: IMPALA-5823: fix SET_DENY_RESERVATION_PROBABILITY ......................................................................
IMPALA-5823: fix SET_DENY_RESERVATION_PROBABILITY Always set the probability even if the client is not yet open. This always worked but added comments explicitly mentioning that it should work. This caused one of the large row tests to start failing with a "failed to repartition" error in the aggregation. The error is a false positive caused by two distinct keys hashing to the same partition. Removing the check allows the query to succeed because the keys hash to different partitions in the next round of repartitioning. Testing: Ran TestSpilling in a loop for a couple of hours, including the exhaustive-only tests. Change-Id: Ib26b697544d6c2312a8e1fe91b0cf8c0917e5603 --- M be/src/exec/exec-node.cc M be/src/exec/exec-node.h M be/src/exec/partitioned-aggregation-node.cc M be/src/exec/partitioned-aggregation-node.h M common/thrift/generate_error_codes.py 5 files changed, 33 insertions(+), 44 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/71/7771/3 -- To view, visit http://gerrit.cloudera.org:8080/7771 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ib26b697544d6c2312a8e1fe91b0cf8c0917e5603 Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]>
