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

(Updated Nov. 29, 2017, 5:23 p.m.)


Review request for sentry, Alexander Kolbasov, kalyan kumar kalvagadda, Na Li, 
Sergio Pena, and Vadim Spector.


Changes
-------

Accidentally added previous version


Repository: sentry


Description
-------

The old retrieveFullPathsImage() method in SentryStore is no longer used by 
actual code (retrieveFullPathsImageUpdate(final String[] prefixes) is used 
instead. It was preserved because it is used by test which now doesn't make 
much sense.


Diffs (updated)
-----

  
sentry-hdfs/sentry-hdfs-service/src/test/java/org/apache/sentry/hdfs/TestImageRetriever.java
 b355630e7 
  
sentry-hdfs/sentry-hdfs-service/src/test/java/org/apache/sentry/hdfs/TestSentryHDFSServiceProcessor.java
 f09d1b228 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
 f32a745ed 
  
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java
 24b11f724 


Diff: https://reviews.apache.org/r/63596/diff/5/

Changes: https://reviews.apache.org/r/63596/diff/4-5/


Testing
-------

mvn -f sentry-provider/sentry-provider-db/pom.xml test -Dtest=TestSentryStore
mvn -f sentry-hdfs/sentry-hdfs-service/pom.xml test 
-Dtest=TestSentryHDFSServiceProcessor
mvn -f sentry-hdfs/sentry-hdfs-service/pom.xml test 
-Dtest=TestImageRetriever.java


Thanks,

Arjun Mishra

Reply via email to