Hello Joe McDonnell, Csaba Ringhofer, Impala Public Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/11123 to look at the new patch set (#3). Change subject: IMPALA-7396: unblock testing with --thread_creation_fault_injection ...................................................................... IMPALA-7396: unblock testing with --thread_creation_fault_injection These are minimal fixes to allow testing new patches with the fault injection. ClientRequestState: fixing this seems somewhat involved, so disabled the fault injection for now to unblock further testing. HdfsScanNode: thread_state_.DecrementNumActive() was called on the error path before the count had actually been incremented. Testing: Ran some of the end-to-end tests for 20-30 minutes and confirmed that no Impalads crashed. start-impala-cluster.py \ --impalad_args=--thread_creation_fault_injection=true impala-py.test tests/query_test/*.py -n2 --verbose Change-Id: Idc9da645c449a266e3a54d0e35b18a2823019cb7 --- M be/src/exec/hdfs-scan-node.cc M be/src/service/client-request-state.cc 2 files changed, 3 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/23/11123/3 -- To view, visit http://gerrit.cloudera.org:8080/11123 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Idc9da645c449a266e3a54d0e35b18a2823019cb7 Gerrit-Change-Number: 11123 Gerrit-PatchSet: 3 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>