Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15046 )
Change subject: IMPALA-9296: Move AuxErrorInfo to StatefulStatus ...................................................................... IMPALA-9296: Move AuxErrorInfo to StatefulStatus This patch moves AuxErrorInfoPB from FragmentInstanceExecStatusPB to StatefulStatusPB. This is necessary because if the report with the AuxErrorInfoPB is dropped (e.g. due to backpressure at the Coordinator or a flaky network), the next report won't contain the AuxErrorInfoPB, and the error info will be lost. StatefulStatus solves this by detecting any reports that may not have been received by the Coordinator, and re-transmitting any StatefulStatuses that were not successfully delivered. This change also makes the setting of AuxErrorInfoPB stateful, so that the error info only shows up in one report and is then dropped from the RuntimeState. Change-Id: Iabbb48dd3ab58ba7b76b1ab6979b92d0e25e72e3 Reviewed-on: http://gerrit.cloudera.org:8080/15046 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/fragment-instance-state.cc M be/src/runtime/runtime-state.cc M be/src/runtime/runtime-state.h M common/protobuf/control_service.proto 5 files changed, 25 insertions(+), 14 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/15046 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iabbb48dd3ab58ba7b76b1ab6979b92d0e25e72e3 Gerrit-Change-Number: 15046 Gerrit-PatchSet: 4 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]>
