Sahil Takiar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14991
Change subject: IMPALA-9267: Fix DCHECK in ClientRequestState::UpdateNonErrorExecState ...................................................................... IMPALA-9267: Fix DCHECK in ClientRequestState::UpdateNonErrorExecState Fixes a DCHECK in ClientRequestState::UpdateNonErrorExecState where the ClientRequestState ExecState attempts to transition from the ERROR to the FINISHED state. The DCHECK was added in IMPALA-6894 in order to prevent any invalid state transition attempts. The fix is to modify UpdateNonErrorExecState so that it skips any attempt to transition from the ERROR to the FINISHED state, which is in line with the behavior prior to IMPALA-6894. Testing: * Ran core tests, unable to reproduce the original issue locally Change-Id: Ie47444ed67704d9469310727eeec2e9a66516e77 --- M be/src/service/client-request-state.cc M be/src/service/client-request-state.h 2 files changed, 21 insertions(+), 12 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/91/14991/1 -- To view, visit http://gerrit.cloudera.org:8080/14991 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ie47444ed67704d9469310727eeec2e9a66516e77 Gerrit-Change-Number: 14991 Gerrit-PatchSet: 1 Gerrit-Owner: Sahil Takiar <[email protected]>
