----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70043/#review213095 -----------------------------------------------------------
sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/HMSPaths.java Lines 507 (patched) <https://reviews.apache.org/r/70043/#comment298962> should this be removed if it has no authz obj? Why change its type to DIR? - Na Li On Feb. 22, 2019, 6:40 p.m., kalyan kumar kalvagadda wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/70043/ > ----------------------------------------------------------- > > (Updated Feb. 22, 2019, 6:40 p.m.) > > > Review request for sentry, Arjun Mishra, Haley Reeve, and Na Li. > > > Bugs: SENTRY-2502 > https://issues.apache.org/jira/browse/SENTRY-2502 > > > Repository: sentry > > > Description > ------- > > Sentry plug-in in name node is stopping to fetch updates from sentry server > when below sequence of events occurs. > > Create a table > Add a single partition to it. > Rename the table created above. > Drop the partition added above > Add the partition again. > Rename the table > > There are a couple of issues observed > > When the table is renamed the Entry object for the table that is renamed is > wrongly update. Type of Entry is changed to "DIR". > When paths have removed the entries with empty paths are not removed from the > collection of Entries for an object. > These two issues can manifest in multiple problems. Issue reported in this > Jira is one such problem. > > > Diffs > ----- > > > sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/HMSPaths.java > 1dfa1d9aafbd482903e4bb935509a76768e02b58 > > sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/HMSPathsDumper.java > 8a7c8a2901af18b375f8637572d9b3ddee0ded6c > > sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryAuthorizationInfo.java > 9fe22ef8cc1533b28462e2b8749e3ba0e7b56582 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationEnd2End.java > 574bc4b0432824abc81fe3c0dd30d1fe01f012e5 > > > Diff: https://reviews.apache.org/r/70043/diff/1/ > > > Testing > ------- > > Added new test case to verify that issues that does not occur after the fix. > > > Thanks, > > kalyan kumar kalvagadda > >
