----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62221/ -----------------------------------------------------------
(Updated Sept. 13, 2017, 8:33 p.m.) Review request for sentry, Alexander Kolbasov, Brian Towles, Na Li, Sergio Pena, and Vamsee Yarlagadda. Changes ------- Using String.format for all inputs Repository: sentry Description ------- Sentry Supportability improvement. 1. HMSFollower * Print confirmation message (at INFO level) once full snapshot is persisted in the DB. * Print the message that HMSFollower is completely ready (after the initial pass of HMSFollower is done) 2. DBUpdateForwarder * Every log message should explicitly mention which type of events is it referring to (PERM or PATH) otherwise there is no way for us to differentiate between calls. Diffs (updated) ----- sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/DBUpdateForwarder.java 8a34d5623 sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryPlugin.java 1318082d3 Diff: https://reviews.apache.org/r/62221/diff/9/ Changes: https://reviews.apache.org/r/62221/diff/8-9/ Testing ------- File Attachments (updated) ---------------- SENTRY-1938.07.patch https://reviews.apache.org/media/uploaded/files/2017/09/13/2f4d17ae-ce15-4b21-b2e0-2e53b4203ac7__SENTRY-1938.07.patch Thanks, Arjun Mishra
