[
https://issues.apache.org/jira/browse/IMPALA-9296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17018506#comment-17018506
]
ASF subversion and git services commented on IMPALA-9296:
---------------------------------------------------------
Commit 7f7743dcc67f112e684737acb5441c544f2c94b1 in impala's branch
refs/heads/master from Sahil Takiar
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=7f7743d ]
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]>
> Move FragmentInstanceExecStatus' AuxErrorInfo to StatefulStatus
> ---------------------------------------------------------------
>
> Key: IMPALA-9296
> URL: https://issues.apache.org/jira/browse/IMPALA-9296
> Project: IMPALA
> Issue Type: Bug
> Components: Backend
> Reporter: Sahil Takiar
> Assignee: Sahil Takiar
> Priority: Major
>
> Surfaced by [~twmarshall] in IMPALA-9295:
> {quote}
> If the report where the AuxErrorInfoPB is sent is dropped/fails, eg. due to
> backpressure at the coordinator or a flaky network, the next report won't
> contain the AuxErrorInfoPB, it will just be dropped.
> Fortunately, we already dealt with this situation for the error log and you
> just need to do the same thing as we did there, i.e. move AuxErrorInfoPB into
> the StatefulStatusPB
> {quote}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
