Hello Lars Volker, Tim Armstrong, Mostafa Mokhtar, Dan Hecht, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9916 to look at the new patch set (#3). Change subject: IMPALA-6792: Fail status reporting if coordinator refuses connections ...................................................................... IMPALA-6792: Fail status reporting if coordinator refuses connections The ReportExecStatusAux() function is run on a dedicated thread per fragment instance. This thread will run until the fragment instance completes executing. On every attempt to send a report to the coordinator, it will attempt to send up to 3 RPCs. If all 3 of them fail, then the fragment instance will cancel itself. However, there is one case where a failure to send the RPC will not be considered a failed RPC. If when we attempt to obtain a new connection, we end up creating a new connection (via ClientCache::CreateClient()) instead of getting a previously cached connection, and this new connection fails to even Open(), it will not be counted as a RPC failure. This patch counts such an error as a failed RPC too. This patch also clarifies some of the error log messages and introduces a flag to control the sleep interval between failed ReportExecStatus RPC retries. Change-Id: If668838f99f78b5ffa713488178b2eb5799ba220 --- M be/src/runtime/query-state.cc 1 file changed, 23 insertions(+), 8 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/16/9916/3 -- To view, visit http://gerrit.cloudera.org:8080/9916 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: If668838f99f78b5ffa713488178b2eb5799ba220 Gerrit-Change-Number: 9916 Gerrit-PatchSet: 3 Gerrit-Owner: Sailesh Mukil <sail...@cloudera.com> Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com> Gerrit-Reviewer: Lars Volker <l...@cloudera.com> Gerrit-Reviewer: Mostafa Mokhtar <mmokh...@cloudera.com> Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>