Wenzhe Zhou has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/16303 )
Change subject: IMPALA-10050: Fixed DCHECK error for backend in terminal state. ...................................................................... IMPALA-10050: Fixed DCHECK error for backend in terminal state. Recent patch for IMPALA-5746 changes the QueryState code and make executer checks if the coordinator is active. If the coordinator is not active, executer cancels the query without calling ReportExecStatus() to send last report. ReportExecStatus() should be called at least once when backend_exec_state is in terminal state so that QueryState::Cancel() will be called if there is execution or RPC error. This patch make QueryState::ReportExecStatus() been called at least once when backend_exec_state is in terminal state. Testing: - Passed TestProcessFailures::test_kill_coordinator. - Passed core tests. Change-Id: Iba6a72f98c0f9299c22c58830ec5a643335b966a --- M be/src/runtime/query-state.cc 1 file changed, 4 insertions(+), 3 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/03/16303/2 -- To view, visit http://gerrit.cloudera.org:8080/16303 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Iba6a72f98c0f9299c22c58830ec5a643335b966a Gerrit-Change-Number: 16303 Gerrit-PatchSet: 2 Gerrit-Owner: Wenzhe Zhou <wz...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Thomas Tauber-Marshall <tmarsh...@cloudera.com>