Re: Review Request 64228: Traverse check in RangerHdfsAuthorizer works incorrectly

2017-12-05 Thread Madhan Neethiraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64228/#review192956 --- Ship it! Ship It! - Madhan Neethiraj On Dec. 5, 2017, 11:45

Re: Review Request 64228: Traverse check in RangerHdfsAuthorizer works incorrectly

2017-12-05 Thread Abhay Kulkarni
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64228/ --- (Updated Dec. 5, 2017, 11:45 p.m.) Review request for ranger, Colm O

Re: Review Request 64228: Traverse check in RangerHdfsAuthorizer works incorrectly

2017-12-05 Thread Madhan Neethiraj
> On Dec. 5, 2017, 9:44 p.m., Madhan Neethiraj wrote: > > hdfs-agent/src/main/java/org/apache/ranger/authorization/hadoop/RangerHdfsAuthorizer.java > > Lines 452 (patched) > > > > > > This treats NOT_DETERMINED as

Re: Review Request 64228: Traverse check in RangerHdfsAuthorizer works incorrectly

2017-12-05 Thread Madhan Neethiraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64228/#review192932 ---

Re: Review Request 64228: Traverse check in RangerHdfsAuthorizer works incorrectly

2017-12-04 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64228/#review192712 --- Ship it! Ship It! - Colm O hEigeartaigh On Dec. 2, 2017,

Re: Review Request 64228: Traverse check in RangerHdfsAuthorizer works incorrectly

2017-12-01 Thread Abhay Kulkarni
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64228/ --- (Updated Dec. 2, 2017, 1:25 a.m.) Review request for ranger, Colm O

Re: Review Request 64228: Traverse check in RangerHdfsAuthorizer works incorrectly

2017-12-01 Thread Zsombor Gegesy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64228/#review192476 --- The 'lack of deny,means allow in traversal' could work as well, I

Re: Review Request 64228: Traverse check in RangerHdfsAuthorizer works incorrectly

2017-12-01 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64228/#review192459 --- LGTM. Zsombor? - Colm O hEigeartaigh On Nov. 30, 2017, 10:38

Review Request 64228: Traverse check in RangerHdfsAuthorizer works incorrectly

2017-11-30 Thread Abhay Kulkarni
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64228/ --- Review request for ranger, Zsombor Gegesy, Madhan Neethiraj, Ramesh Mani, and