----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45859/ -----------------------------------------------------------
(Updated Oct. 7, 2016, 12:38 a.m.) Review request for sentry, Alexander Kolbasov, Anne Yu, Hao Hao, Lenni Kuff, and Sravya Tirukkovalur. Repository: sentry Description ------- Show role / privileges info in Sentry Service Webpage. Since it is only used for debug / test currently, this webpage can be seen only when SENTRY_WEB_ADMIN_SERVLET_ENABLED is true. Diffs (updated) ----- sentry-service/sentry-service-common/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java 5ec364c460e74d0a9dae8a28c20042360157b8a0 sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryAdminServlet.java PRE-CREATION sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryWebServer.java a42f395270996da345ce49edca909e0438383759 Diff: https://reviews.apache.org/r/45859/diff/ Testing ------- Already tested in kerberos cluster. When sentry.service.web.authentication.type is set to KERBEROS, only the SENTRY_WEB_SECURITY_ALLOW_CONNECT_USERS can see this page. Also this webpage can be seen only when SENTRY_WEB_ADMIN_SERVLET_ENABLED is true. Thanks, Li Li