----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43464/ -----------------------------------------------------------
(Updated Feb. 18, 2016, 6:34 p.m.) Review request for sentry, Anne Yu, Hao Hao, Lenni Kuff, and Sravya Tirukkovalur. Changes ------- Remove SentryServiceServlet, and reuse original AdminServlet. Repository: sentry Description ------- Use the twitter Bootstrap kit (or similar) to beautify the Sentry Service webpage Diffs (updated) ----- sentry-provider/sentry-provider-db/pom.xml 38e0924f18383b3f0186e4810db3e1a057c2f6b6 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryWebServer.java fdb99ce091d59fc475f35c4cc263975cb909120e sentry-provider/sentry-provider-db/src/main/webapp/SentryService.html PRE-CREATION sentry-provider/sentry-provider-db/src/main/webapp/css/bootstrap-theme.min.css PRE-CREATION sentry-provider/sentry-provider-db/src/main/webapp/css/bootstrap.min.css PRE-CREATION sentry-provider/sentry-provider-db/src/main/webapp/css/sentry.css PRE-CREATION sentry-provider/sentry-provider-db/src/main/webapp/sentry.png PRE-CREATION Diff: https://reviews.apache.org/r/43464/diff/ Testing ------- The prejenkins job has some issue with the png file (sentry logo) in the patch, so I removed the sentry logo when submiting the patch there. Thanks, Li Li
