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

(Updated Sept. 7, 2017, 3:43 p.m.)


Review request for sentry, Alexander Kolbasov, Arjun Mishra, Na Li, Sergio 
Pena, and Vadim Spector.


Changes
-------

Converted to a Singleton for state notification


Bugs: SENTRY-1918
    https://issues.apache.org/jira/browse/SENTRY-1918


Repository: sentry


Description (updated)
-------

Check the state if the SentryService to see if its doing a full update and 
delay the NN requests for a full update while thats going on.


Diffs (updated)
-----

  
sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/DBUpdateForwarder.java
 5d744214e14d6c48194b3a0bf84d6e10070b020a 
  
sentry-hdfs/sentry-hdfs-service/src/test/java/org/apache/sentry/hdfs/TestDBUpdateForwarder.java
 8fbc10048003ab4b8a38676e241ae0fd27d2392c 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java
 c234eafb430b58bfd1e739847e83937a5e34d878 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollowerState.java
 PRE-CREATION 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java
 d44abffc199cd46d3ab7d2230dfdb2075736a8f0 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryServiceState.java
 PRE-CREATION 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryState.java
 PRE-CREATION 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryStateBank.java
 PRE-CREATION 
  
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/SentryStateBankTestHelper.java
 PRE-CREATION 
  
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestSentryStateBank.java
 PRE-CREATION 


Diff: https://reviews.apache.org/r/62107/diff/6/

Changes: https://reviews.apache.org/r/62107/diff/5-6/


Testing
-------

Unit Tests


Thanks,

Brian Towles

Reply via email to