----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57570/ -----------------------------------------------------------
(Updated March 22, 2017, 2:27 p.m.) Review request for sentry, Alexander Kolbasov and Hao Hao. Changes ------- Re-work the patch against the master branch, and add integration test as Hao suggested. Bugs: SENTRY-1644 https://issues.apache.org/jira/browse/SENTRY-1644 Repository: sentry Description ------- After rename Hive table with partitions, ACLs on the partition paths disappear. It is caused by the fact that renaming on sentry does not move any sub-namespace. Diffs (updated) ----- sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/HMSPaths.java 6d2ab23c3f65af5430ea02563e13c4c4c49aa1c6 sentry-hdfs/sentry-hdfs-common/src/test/java/org/apache/sentry/hdfs/TestHMSPaths.java 0dee1028ac852b12b109e875b2cbbb0bd2efbe03 sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationAdvanced.java d079628a0e549e3b179c7a321f82bcf6b580da43 Diff: https://reviews.apache.org/r/57570/diff/2/ Changes: https://reviews.apache.org/r/57570/diff/1-2/ Testing ------- mvn test -Dtest=TestHMSPaths Thanks, Lei Xu