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>

Reply via email to