Hello Michael Ho, Pooja Nilangekar, Tim Armstrong, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/11369

to look at the new patch set (#2).

Change subject: IMPALA-7517. Fix hang in scanner threads when soft limit is 
exceeded
......................................................................

IMPALA-7517. Fix hang in scanner threads when soft limit is exceeded

As described in the JIRA, when scanner threads see that the soft limit
has been exceeded, they try to shut down. In some particular
interleavings, this would cause all of the scanner threads to exit
without any of them marking the scan as completed.

This patch adds a new fault point to inject fake soft limit errors, and
adds this fault point to the scanner test. With the previous placement
of the soft limit check, this caused query hangs pretty reliably. With
the new placement of the memory limit check, it now passes.

Change-Id: I3dc1a2ec79c823575d7d40e7b52216dea5b0ddde
---
M be/src/exec/hdfs-scan-node.cc
M tests/query_test/test_scanners.py
2 files changed, 12 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/69/11369/2
--
To view, visit http://gerrit.cloudera.org:8080/11369
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3dc1a2ec79c823575d7d40e7b52216dea5b0ddde
Gerrit-Change-Number: 11369
Gerrit-PatchSet: 2
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Ho <[email protected]>
Gerrit-Reviewer: Pooja Nilangekar <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>

Reply via email to