----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64545/ -----------------------------------------------------------
(Updated Dec. 13, 2017, 5:34 p.m.) Review request for sentry, Brian Towles, kalyan kumar kalvagadda, Na Li, Sergio Pena, and Vadim Spector. Changes ------- Including Vadim and Sasha's suggestions Repository: sentry Description ------- Currently sentry's log messaging when it becomes a 'writer' (the sentry server that reads from HMS) is not obvious, and is really mostly discernable at the debug level. Add a log message at the INFO level, such as 'This sentry server has just become the HMS reader/writer' that is printed once when the sentry server first becomes the HMS reader/writer. Diffs (updated) ----- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/LeaderStatusMonitor.java 360c5a530 Diff: https://reviews.apache.org/r/64545/diff/4/ Changes: https://reviews.apache.org/r/64545/diff/3-4/ Testing ------- mvn -f sentry-provider/sentry-provider-db/pom.xml test Thanks, Arjun Mishra