impala-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Ho (JIRA)" <>
Subject [jira] [Created] (IMPALA-5558) Retry loop in QueryState::ReportExecStatusAux() is ineffective
Date Thu, 22 Jun 2017 07:04:00 GMT
Michael Ho created IMPALA-5558:

             Summary: Retry loop in QueryState::ReportExecStatusAux() is ineffective
                 Key: IMPALA-5558
             Project: IMPALA
          Issue Type: Bug
          Components: Distributed Exec
    Affects Versions: Impala 2.9.0
            Reporter: Michael Ho
            Assignee: Michael Ho
            Priority: Minor

The following loop aims to retry the RPC for up to 3 times when reporting exec status of a
fragment instance to the coordinator. However, it's not very effective because the client
connection isn't re-established between retry. In addition, since we are reporting the query
profile, it should be fine to retry all the time even if the payload was partially sent to
the remote client.

cc'ing [~henryr] and [~sailesh]

  // Try to send the RPC 3 times before failing.
  for (int i = 0; i < 3; ++i) {
    rpc_status = coord.DoRpc(
        &ImpalaBackendClient::ReportExecStatus, params, &res, &retry_is_safe);
    if (rpc_status.ok()) break;
    if (!retry_is_safe) break;
    if (i < 2) SleepForMs(RETRY_SLEEP_MS);

This message was sent by Atlassian JIRA

View raw message