Attila Jeges has uploaded a new patch set (#2). ( 
http://gerrit.cloudera.org:8080/16573 )

Change subject: IMPALA-10224: Add startup flag not to expose debug web url to 
clients
......................................................................

IMPALA-10224: Add startup flag not to expose debug web url to clients

This patch introduces a new startup flag
--ping_expose_webserver_url (true by default) to control whether
PingImpalaService, PingImpalaHS2Service RPC calls should expose
the debug web url to the client or not.

This is necessary as the debug web UI is not something that
end-users will necessarily have access to.

If the flag is set to false, the RPC calls will return an empty
string instead of the real url signalling that the debug web ui
is not available.

Note that if the webserver is disabled (--enable_webserver flag
is set to false) the RPC calls will behave the same and return an
empty string for the url.

Change-Id: I7ec3e92764d712b8fee63c1f45b038c31c184cfc
---
M be/src/runtime/exec-env.cc
M be/src/runtime/exec-env.h
M be/src/service/impala-beeswax-server.cc
M be/src/service/impala-hs2-server.cc
M shell/impala_client.py
M shell/impala_shell.py
M tests/custom_cluster/test_web_pages.py
7 files changed, 62 insertions(+), 15 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/73/16573/2
--
To view, visit http://gerrit.cloudera.org:8080/16573
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I7ec3e92764d712b8fee63c1f45b038c31c184cfc
Gerrit-Change-Number: 16573
Gerrit-PatchSet: 2
Gerrit-Owner: Attila Jeges <[email protected]>
Gerrit-Reviewer: Attila Jeges <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>

Reply via email to