Hello Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/15027
to look at the new patch set (#2).
Change subject: IMPALA-9295: Read aux error info regardless of overall query
status
......................................................................
IMPALA-9295: Read aux error info regardless of overall query status
IMPALA-9137 added AuxErrorInfoPB to FragmentInstanceExecStatusPB
in order to propagate additional error information from Impala executors
to coordinators. The extra error information helps the coordinator decide
if it should blacklist any nodes.
Prior to this patch, the aux error info was only read if the
overall_status of the ReportExecStatusRequestPB was an error. However,
Impala executors don't synchronize the setting of the the aux error info
and the overall_status, so it is possible the aux error info is dropped.
This patch fixes this issue by always looking for the aux error info
from each fragment, regardless of the overall_status from the report.
Testing:
* Core tests pass
Change-Id: I7dfb5efd7a70626dfc5227f28288ff52ebb4861a
---
M be/src/runtime/coordinator-backend-state.cc
M be/src/runtime/coordinator-backend-state.h
M be/src/runtime/coordinator.cc
M be/src/runtime/coordinator.h
4 files changed, 28 insertions(+), 19 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/27/15027/2
--
To view, visit http://gerrit.cloudera.org:8080/15027
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I7dfb5efd7a70626dfc5227f28288ff52ebb4861a
Gerrit-Change-Number: 15027
Gerrit-PatchSet: 2
Gerrit-Owner: Sahil Takiar <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
