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

(Updated April 26, 2017, 6:51 a.m.)


Review request for sentry, Hao Hao, kalyan kumar kalvagadda, Na Li, Sergio 
Pena, Vamsee Yarlagadda, and Vadim Spector.


Changes
-------

merged with SENTRY-1587 changes.
This involved changes to Notification processor in the way it handles 
PathsUpdate.parsePath()


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


Repository: sentry


Description
-------

SENTRY-1687 FullUpdateInitializer can be more efficient


Diffs (updated)
-----

  
sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/FullUpdateInitializer.java
 d876d230f3200c71e9d11681efce8b24cac40b7e 
  
sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/PathsUpdate.java
 e32d4a720e49464f812b12c46e8e0e040cb36fad 
  
sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/ServiceConstants.java
 23552c2512902a8500bfacb1c745ca4b10498cc8 
  
sentry-hdfs/sentry-hdfs-common/src/test/java/org/apache/sentry/hdfs/TestFullUpdateInitializer.java
 f338ce8abddcde128536a0efef77983990325aa6 
  
sentry-hdfs/sentry-hdfs-common/src/test/java/org/apache/sentry/hdfs/TestPathsUpdate.java
 b5cbea9d295385bb38b912c13903edace04d7589 
  
sentry-hdfs/sentry-hdfs-common/src/test/java/org/apache/sentry/hdfs/TestUpdateableAuthzPaths.java
 e643e01a45de77ef7f465d6921c5ae9e7ce925a2 
  
sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/PathImageRetriever.java
 0eaac80edcb4434d5f8953a44050720c5770faf3 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
 8b88c9a2dbbb471d15e177f3d0a129256a19f228 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java
 57b7f888d1eb9e10447fe3190579aec6e05b661c 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/NotificationProcessor.java
 84574f0fca4596d475aed48797a2d4d6fad02915 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationAdvanced.java
 e771ce7de5a6849f48a036b2b1c30b435d8cb520 


Diff: https://reviews.apache.org/r/58284/diff/3/

Changes: https://reviews.apache.org/r/58284/diff/2-3/


Testing
-------

Updated the unit test to test for bigger HMS snapshots


Thanks,

Alexander Kolbasov

Reply via email to