----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57570/#review171020 -----------------------------------------------------------
Fix it, then Ship it! Ship It! sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/HMSPaths.java Lines 112 (patched) <https://reviews.apache.org/r/57570/#comment243852> Each individual entry sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/HMSPaths.java Lines 114 (patched) <https://reviews.apache.org/r/57570/#comment243853> Please axtend this comment to show how the tree is represented (a small example would help) - it isn't clear what is rthe relationship with Entry and the HDFS path. That aside - the full hdfs path can contain uri prefix - is it always trimmed somewhere before it reaches here? sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/HMSPaths.java Lines 339 (patched) <https://reviews.apache.org/r/57570/#comment243851> should it be private? The return value is never used - is it intended? - Alexander Kolbasov On March 28, 2017, 12:24 a.m., Lei Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57570/ > ----------------------------------------------------------- > > (Updated March 28, 2017, 12:24 a.m.) > > > Review request for sentry, Alexander Kolbasov and Hao Hao. > > > 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 > ----- > > > 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/3/ > > > Testing > ------- > > mvn test -Dtest=TestHMSPaths > > > File Attachments > ---------------- > > SENTRY-1644.003-master.patch > > https://reviews.apache.org/media/uploaded/files/2017/03/28/1b82d5bc-0267-42d3-9a9a-9515ed7cb25e__SENTRY-1644.003-master.patch > > > Thanks, > > Lei Xu > >