Re: Review Request 63645: SENTRY-2032: Leading Slashes need to removed when creating HMS path entries

2017-11-16 Thread Vadim Spector via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63645/#review191247 --- Ship it! Ship It! - Vadim Spector On Nov. 16, 2017, 8:38

Re: Review Request 63645: SENTRY-2032: Leading Slashes need to removed when creating HMS path entries

2017-11-16 Thread Arjun Mishra via Review Board
> On Nov. 16, 2017, 7:11 p.m., Vadim Spector wrote: > > sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/utils/PathUtils.java > > Line 220 (original), 225 (patched) > > > > > > at this

Re: Review Request 63645: SENTRY-2032: Leading Slashes need to removed when creating HMS path entries

2017-11-16 Thread Vadim Spector via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63645/#review191214 ---

Re: Review Request 63645: SENTRY-2032: Leading Slashes need to removed when creating HMS path entries

2017-11-08 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63645/ --- (Updated Nov. 8, 2017, 2:59 p.m.) Review request for sentry, Sergio Pena and

Re: Review Request 63645: SENTRY-2032: Leading Slashes need to removed when creating HMS path entries

2017-11-08 Thread Arjun Mishra via Review Board
> On Nov. 8, 2017, 7 a.m., Alexander Kolbasov wrote: > > Do you also want to update PathUpdate.parsePath() to be consistent? I will check with Vadim on that. > On Nov. 8, 2017, 7 a.m., Alexander Kolbasov wrote: > >

Re: Review Request 63645: SENTRY-2032: Leading Slashes need to removed when creating HMS path entries

2017-11-07 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63645/#review190428 --- Do you also want to update PathUpdate.parsePath() to be

Re: Review Request 63645: SENTRY-2032: Leading Slashes need to removed when creating HMS path entries

2017-11-07 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63645/#review190431 ---

Review Request 63645: SENTRY-2032: Leading Slashes need to removed when creating HMS path entries

2017-11-07 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63645/ --- Review request for sentry, Sergio Pena and Vadim Spector. Repository: sentry