Re: Review Request 67899: SENTRY-2299: NPE In Sentry HDFS Sync Plugin

2018-07-13 Thread Na Li via Review Board
> On July 13, 2018, 3:35 p.m., kalyan kumar kalvagadda wrote: > > sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/HMSPaths.java > > Lines 881-885 (patched) > > > > > > Better way to avoid this

Re: Review Request 67899: SENTRY-2299: NPE In Sentry HDFS Sync Plugin

2018-07-13 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67899/ --- (Updated July 13, 2018, 3:45 p.m.) Review request for sentry, Arjun Mishra,

Re: Review Request 67899: SENTRY-2299: NPE In Sentry HDFS Sync Plugin

2018-07-13 Thread Na Li via Review Board
> On July 13, 2018, 1:16 p.m., kalyan kumar kalvagadda wrote: > > From your comment, "Sentry client at NameNode does not create entry for > > external table. When user rename an external table with different path, > > null exception happens" > > > > Does does that mean sentry ACL's don't work

Re: Review Request 67899: SENTRY-2299: NPE In Sentry HDFS Sync Plugin

2018-07-13 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67899/#review206055 ---

Re: Review Request 67899: SENTRY-2299: NPE In Sentry HDFS Sync Plugin

2018-07-13 Thread kalyan kumar kalvagadda via Review Board
> On July 13, 2018, 1:16 p.m., kalyan kumar kalvagadda wrote: > > From your comment, "Sentry client at NameNode does not create entry for > > external table. When user rename an external table with different path, > > null exception happens" > > > > Does does that mean sentry ACL's don't work

Re: Review Request 67899: SENTRY-2299: NPE In Sentry HDFS Sync Plugin

2018-07-13 Thread kalyan kumar kalvagadda via Review Board
> On July 13, 2018, 1:16 p.m., kalyan kumar kalvagadda wrote: > > From your comment, "Sentry client at NameNode does not create entry for > > external table. When user rename an external table with different path, > > null exception happens" > > > > Does does that mean sentry ACL's don't work

Re: Review Request 67899: SENTRY-2299: NPE In Sentry HDFS Sync Plugin

2018-07-13 Thread Na Li via Review Board
> On July 13, 2018, 1:16 p.m., kalyan kumar kalvagadda wrote: > > From your comment, "Sentry client at NameNode does not create entry for > > external table. When user rename an external table with different path, > > null exception happens" > > > > Does does that mean sentry ACL's don't work

Re: Review Request 67899: SENTRY-2299: NPE In Sentry HDFS Sync Plugin

2018-07-13 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67899/#review206050 --- >From your comment, "Sentry client at NameNode does not create

Re: Review Request 67899: SENTRY-2299: NPE In Sentry HDFS Sync Plugin

2018-07-12 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67899/#review206033 --- Ship it! Ship It! - Sergio Pena On July 12, 2018, 9:14

Re: Review Request 67899: SENTRY-2299: NPE In Sentry HDFS Sync Plugin

2018-07-12 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67899/#review206025 --- Ship it! Ship It! - Arjun Mishra On July 12, 2018, 9:14

Re: Review Request 67899: SENTRY-2299: NPE In Sentry HDFS Sync Plugin

2018-07-12 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67899/ --- (Updated July 12, 2018, 9:14 p.m.) Review request for sentry, Arjun Mishra,