Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15027 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/15027 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- 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(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: I7dfb5efd7a70626dfc5227f28288ff52ebb4861a Gerrit-Change-Number: 15027 Gerrit-PatchSet: 5 Gerrit-Owner: Sahil Takiar <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Sahil Takiar <[email protected]> Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]>
