Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/21128 )

Change subject: IMPALA-12264: Add limit on number of HS2 sessions per user.
......................................................................

IMPALA-12264: Add limit on number of HS2 sessions per user.

Add a new flag --max_hs2_sessions_per_user which sets a limit on the
number of Hiveserver2 sessions that can be concurrently opened by a
user on a coordinator. By default this value is -1, which disables
the new feature. An attempt to open more sessions than the new flag
value results in an error. This is implemented by maintaining a map of
users to a count of HS2 sessions.

If the per-user HS2 session counts are being limited in this way,
then the per-user counts are visible on the /sessions page of the
webui.

Add a new test case in test_session_expiration.py.

Change-Id: Idd28edc352102d89774f6ece5376e7c79ae41aa8
Reviewed-on: http://gerrit.cloudera.org:8080/21128
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
M be/src/service/impala-hs2-server.cc
M be/src/service/impala-http-handler.cc
M be/src/service/impala-http-handler.h
M be/src/service/impala-server.cc
M be/src/service/impala-server.h
M tests/custom_cluster/test_session_expiration.py
M www/sessions.tmpl
7 files changed, 184 insertions(+), 30 deletions(-)

Approvals:
  Impala Public Jenkins: Looks good to me, approved; Verified

--
To view, visit http://gerrit.cloudera.org:8080/21128
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Idd28edc352102d89774f6ece5376e7c79ae41aa8
Gerrit-Change-Number: 21128
Gerrit-PatchSet: 8
Gerrit-Owner: Andrew Sherman <[email protected]>
Gerrit-Reviewer: Abhishek Rawat <[email protected]>
Gerrit-Reviewer: Andrew Sherman <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Xiang Yang <[email protected]>

Reply via email to