----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57570/#review168939 -----------------------------------------------------------
Can you add test cases in the intergation test as well, TestHDFSIntegrationAdvanced. To ensure cases such as for managed/unmanaged table we get expected output. Also, it looks like the tests are not all green. Would you mind take a look if anything related? - Hao Hao On March 13, 2017, 9:57 p.m., Lei Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57570/ > ----------------------------------------------------------- > > (Updated March 13, 2017, 9:57 p.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 > e4850f4ee039f011de370ca6b2618161150645de > > sentry-hdfs/sentry-hdfs-common/src/test/java/org/apache/sentry/hdfs/TestHMSPaths.java > 0dee1028ac852b12b109e875b2cbbb0bd2efbe03 > > > Diff: https://reviews.apache.org/r/57570/diff/1/ > > > Testing > ------- > > mvn test -Dtest=TestHMSPaths > > > Thanks, > > Lei Xu > >
