Re: Sentry-jdk-1.8 - Build # 56 - Failure

2018-06-20 Thread Kalyan Kumar Kalvagadda
Hello all, Is any one looking into this failure? On Wed, Jun 20, 2018 at 10:30 AM Apache Jenkins Server < jenk...@builds.apache.org> wrote: > The Apache Jenkins build system has built Sentry-jdk-1.8 (build #56) > > Status: Failure > > Check console output at

Re: Review Request 67648: SENTRY-2251: Update user privileges based on changes to authorizables

2018-06-20 Thread kalyan kumar kalvagadda via Review Board
> On June 20, 2018, 2:12 p.m., Arjun Mishra wrote: > > Looks good. Have asked for some clarification. I responded to them. - kalyan kumar --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 67658: SENTRY-2269: Make SentryStore pluggable

2018-06-20 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67658/#review205095 ---

Re: Review Request 67658: SENTRY-2269: Make SentryStore pluggable

2018-06-20 Thread kalyan kumar kalvagadda via Review Board
> On June 20, 2018, 4:32 p.m., kalyan kumar kalvagadda wrote: > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStoreInterface.java > > Lines 57-81 (patched) > >

Re: Review Request 67658: SENTRY-2269: Make SentryStore pluggable

2018-06-20 Thread Fahd Siddiqui
> On June 20, 2018, 4:32 p.m., kalyan kumar kalvagadda wrote: > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStoreInterface.java > > Lines 51-52 (patched) > >

Re: Review Request 67658: SENTRY-2269: Make SentryStore pluggable

2018-06-20 Thread Fahd Siddiqui
> On June 20, 2018, 4:42 p.m., kalyan kumar kalvagadda wrote: > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/classification/InterfaceAudience.java > > Lines 37-45 (patched) > >

Re: Review Request 67658: SENTRY-2269: Make SentryStore pluggable

2018-06-20 Thread Fahd Siddiqui
> On June 20, 2018, 4:32 p.m., kalyan kumar kalvagadda wrote: > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStoreInterface.java > > Lines 57-81 (patched) > >

Re: Review Request 67658: SENTRY-2269: Make SentryStore pluggable

2018-06-20 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67658/#review205093 --- Fix it

Sentry-jdk-1.8 - Build # 56 - Failure

2018-06-20 Thread Apache Jenkins Server
The Apache Jenkins build system has built Sentry-jdk-1.8 (build #56) Status: Failure Check console output at https://builds.apache.org/job/Sentry-jdk-1.8/56/ to view the results.

Re: Review Request 67648: SENTRY-2251: Update user privileges based on changes to authorizables

2018-06-20 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67648/#review205074 ---

Re: Review Request 67648: SENTRY-2251: Update user privileges based on changes to authorizables

2018-06-20 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67648/#review205088 ---

Re: Review Request 67646: SENTRY-2272: Fix the sentry store logic for listing user privileges

2018-06-20 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67646/#review205091 ---

Re: Review Request 67648: SENTRY-2251: Update user privileges based on changes to authorizables

2018-06-20 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67648/ --- (Updated June 20, 2018, 1:27 p.m.) Review request for sentry, Arjun Mishra, Na

Re: Review Request 67648: SENTRY-2251: Update user privileges based on changes to authorizables

2018-06-20 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67648/#review205077 --- Ship it! Looks good. Have asked for some clarification. -

Re: Review Request 67649: SENTRY-2274: Grant and revoke owner privileges based on HMS updates(server-side)

2018-06-20 Thread kalyan kumar kalvagadda via Review Board
> On June 19, 2018, 10:38 p.m., Sergio Pena wrote: > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyStoreProcessor.java > > Lines 1386 (patched) > > > > > >