tree 11d3cee70f14e7bc82d4ef1429fdcd39a01f5afc
parent df13de70fe5a286d7d3596bf2f1ecd71945e8f27
author Sahil Takiar <takiar.sahil@gmail.com> 1579123720 -0800
committer Impala Public Jenkins <impala-public-jenkins@cloudera.com> 1579313145 +0000

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 <impala-public-jenkins@cloudera.com>
Tested-by: Impala Public Jenkins <impala-public-jenkins@cloudera.com>