Andrew Sherman has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12102
Change subject: IMPALA-7183: Include remote host when logging unknown execution status. ...................................................................... IMPALA-7183: Include remote host when logging unknown execution status. ControlService::ReportExecStatus reports the execution status of a fragment to the coordinator. When the coordinator receives a ReportExecStatus call, it looks up the ClientRequestState for the query id. If this cannot be found, usually because the query is in the process of being canceled, then it logs a message before responding to the RPC. Add the remote host name to this message. We try to use the DNS reverse-lookup name, if that fails we use the IP address instead. Example output: "ReportExecStatus(): Received report for unknown query ID (probably closed or cancelled): 9a437a10d83cfd86:3c96113600000000 remote host=localhost" TESTING Ran end-to-end tests. Forced code to execute by hand to check output is correct. Change-Id: I79b223e22096e2d964b72654a34dc0d88e8c6422 --- M be/src/service/control-service.cc 1 file changed, 9 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/02/12102/1 -- To view, visit http://gerrit.cloudera.org:8080/12102 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I79b223e22096e2d964b72654a34dc0d88e8c6422 Gerrit-Change-Number: 12102 Gerrit-PatchSet: 1 Gerrit-Owner: Andrew Sherman <[email protected]> Gerrit-Reviewer: Thomas Marshall <[email protected]>
