Re: Review Request 67131: SENTRY-2174: Sentry authorization provider should now generate ACL for users

2018-05-17 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67131/#review203347 --- Ship it! Ship It! - Sergio Pena On May 16, 2018, 6:41 p.m.,

Re: Review Request 67131: SENTRY-2174: Sentry authorization provider should now generate ACL for users

2018-05-16 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67131/#review203285 --- Ship it! Ship It! - Na Li On May 16, 2018, 6:41 p.m.,

Re: Review Request 67131: SENTRY-2174: Sentry authorization provider should now generate ACL for users

2018-05-16 Thread kalyan kumar kalvagadda via Review Board
> On May 16, 2018, 3:52 p.m., Na Li wrote: > > sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryPermissions.java > > Lines 278 (patched) > > > > > > Is it abnormal for this to

Re: Review Request 67131: SENTRY-2174: Sentry authorization provider should now generate ACL for users

2018-05-16 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67131/#review203240 ---

Re: Review Request 67131: SENTRY-2174: Sentry authorization provider should now generate ACL for users

2018-05-16 Thread kalyan kumar kalvagadda via Review Board
> On May 16, 2018, 3:52 p.m., Na Li wrote: > > sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryPermissions.java > > Lines 245 (patched) > > > > > > should you check the type if

Re: Review Request 67131: SENTRY-2174: Sentry authorization provider should now generate ACL for users

2018-05-16 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67131/#review203235 ---

Re: Review Request 67131: SENTRY-2174: Sentry authorization provider should now generate ACL for users

2018-05-15 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67131/ --- (Updated May 15, 2018, 11:45 p.m.) Review request for sentry, Arjun Mishra, Na

Re: Review Request 67131: SENTRY-2174: Sentry authorization provider should now generate ACL for users

2018-05-15 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67131/#review203138 ---

Review Request 67131: SENTRY-2174: Sentry authorization provider should now generate ACL for users

2018-05-15 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67131/ --- Review request for sentry, Arjun Mishra, Na Li, and Sergio Pena. Bugs: