Vincent Tran has uploaded a new patch set (#9). ( 
http://gerrit.cloudera.org:8080/14782 )

Change subject: IMPALA-9182: Print the socket address of the client closing a 
session or cancelling a query from the WebUI
......................................................................

IMPALA-9182: Print the socket address of the client closing a session or 
cancelling a query from the WebUI

This change appends the socket address (HOST:PORT) of the client
who made the request to close a session or cancel a query from
the coordinator's debug WebUI.

Existing statuses:
"Cancelled from Impala's debug web interface"
"Session closed from Impala's debug web interface"

New statuses:
"Cancelled from Impala's debug web interface by client at
 <host>:<port>"
"Session closed from Impala's debug web interface by client
 at <host>:<port>"

Testing:
-Verified visually that the status message is printed in the impalad
 log with the socket address when one cancels a query or closes a session.
-Added a new e2e test to verify that the new status gets printed in
 runtime profile and coordinator log when a query is cancelled in this
 way.
-Made log asserts more robust by adding a timeout/wait value.

Change-Id: Icf74ad06ce1c40fab4ce37de6b7ca78e3e520b43
---
M be/src/kudu/util/web_callback_registry.h
M be/src/service/impala-http-handler.cc
M be/src/util/webserver.cc
M tests/common/impala_test_suite.py
M tests/observability/test_log_fragments.py
M tests/webserver/test_web_pages.py
6 files changed, 76 insertions(+), 30 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/82/14782/9
--
To view, visit http://gerrit.cloudera.org:8080/14782
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Icf74ad06ce1c40fab4ce37de6b7ca78e3e520b43
Gerrit-Change-Number: 14782
Gerrit-PatchSet: 9
Gerrit-Owner: Vincent Tran <vtt...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Reviewer: Vincent Tran <vtt...@cloudera.com>

Reply via email to