-----------------------------------------------------------
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

Reply via email to