Re: Review Request 63086: SENTRY-1993: StringIndexOutOfBoundsException in HMSPathsDumper.java

2017-10-20 Thread Vadim Spector via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63086/#review188866 --- Ship it! New patch added call to

Re: Review Request 63086: SENTRY-1993: StringIndexOutOfBoundsException in HMSPathsDumper.java

2017-10-20 Thread Misha Dmitriev via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63086/ --- (Updated Oct. 20, 2017, 7:14 p.m.) Review request for sentry, Alexander

Re: questions on sentry

2017-10-20 Thread Alexander Kolbasov
To answer your second question - IP address based authorization isn't supported. - Alex On Thu, Oct 19, 2017 at 8:20 AM Chen Song wrote: > Thanks Alexander. > > I asked because I see the HDFS ACLs sync feature for Sentry. I am clear > now. > Do you have any idea on my

Review Request 63185: SENTRY-2013 - Align the SentryGenericServiceClient and SentryPolicyServiceClient a bit more closely

2017-10-20 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63185/ --- Review request for sentry. Bugs: SENTRY-2013

Review Request 63178: SENTRY-1989: Bump Kafka version from 0.9 to 0.11

2017-10-20 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63178/ --- Review request for sentry, Na Li and Sergio Pena. Bugs: SENTRY-1989

Re: Review Request 63160: SENTRY-1667: Switching to Jetty v9 library

2017-10-20 Thread Colm O hEigeartaigh
> On Oct. 19, 2017, 7:25 p.m., kalyan kumar kalvagadda wrote: > > I'm fine with bumping up the jetty version but the only concern is that the > > Solr tests will be failing unitl changes for SENTRY-1475 are committed. Agreed, probably best waiting until the Solr upgrade happens, assuming of