-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62221/
-----------------------------------------------------------

(Updated Sept. 13, 2017, 6:42 p.m.)


Review request for sentry, Alexander Kolbasov, Brian Towles, Na Li, Sergio 
Pena, and Vamsee Yarlagadda.


Changes
-------

Restricting changes to only logging


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/8/

Changes: https://reviews.apache.org/r/62221/diff/7-8/


Testing
-------


Thanks,

Arjun Mishra

Reply via email to