----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63596/ -----------------------------------------------------------
(Updated Nov. 27, 2017, 6 p.m.) Review request for sentry, Alexander Kolbasov, Sergio Pena, and Vadim Spector. Changes ------- Updateed patch after SENTRY-2032 was committed 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/2/ Changes: https://reviews.apache.org/r/63596/diff/1-2/ 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