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

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

Re: Review Request 67134: SENTRY-2224: Support SHOW GRANT on HIVE_OBJECT

2018-06-19 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67134/ --- (Updated June 19, 2018, 7:43 p.m.) Review request for sentry, kalyan kumar

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

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

Re: Review Request 67134: SENTRY-2224: Support SHOW GRANT on HIVE_OBJECT

2018-06-19 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67134/ --- (Updated June 19, 2018, 7:46 p.m.) Review request for sentry, kalyan kumar

Re: Review Request 67652: SENTRY-2271: Wrong message seen when db upgrade failure.

2018-06-19 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67652/#review205029 --- Ship it! Ship It! - Sergio Pena On June 19, 2018, 9:52

Re: Review Request 67650: SENTRY-2275: Grant and revoke owner privileges based on HMS updates(client-side)

2018-06-19 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67650/#review205030 --- Ship it! Ship It! - Sergio Pena On June 19, 2018, 9:11

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

2018-06-19 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 19, 2018, 10:22 p.m.) Review request for sentry, Arjun Mishra,

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

2018-06-19 Thread kalyan kumar kalvagadda via Review Board
> On June 19, 2018, 6:47 p.m., Arjun Mishra wrote: > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java > > Lines 797 (patched) > > > > > > Why

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

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

Re: Review Request 67134: SENTRY-2224: Support SHOW GRANT on HIVE_OBJECT

2018-06-19 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67134/#review205035 --- Ship it! Ship It! - Sergio Pena On June 19, 2018, 7:46

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

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

Review Request 67650: SENTRY-2241: Extend the Sync Listener to pass owner information to sentry server.(client side)

2018-06-19 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67650/ --- Review request for sentry, Na Li and Sergio Pena. Bugs: SENTRY-2241

Re: Review Request 67650: SENTRY-2241: Extend the Sync Listener to pass owner information to sentry server.(client side)

2018-06-19 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67650/ --- (Updated June 19, 2018, 4:13 p.m.) Review request for sentry, Na Li and Sergio

Review Request 67652: SENTRY-2271: Wrong message seen when db upgrade failure.

2018-06-19 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67652/ --- Review request for sentry and Sergio Pena. Repository: sentry Description

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

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

Re: Review Request 67134: SENTRY-2224: Support SHOW GRANT on HIVE_OBJECT

2018-06-19 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67134/ --- (Updated June 19, 2018, 4:04 p.m.) Review request for sentry, kalyan kumar

Re: Review Request 67652: SENTRY-2271: Wrong message seen when db upgrade failure.

2018-06-19 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67652/#review205004 --- Ship it! Ship It! - Sergio Pena On June 19, 2018, 4:25

Re: Review Request 67560: SENTRY-2241: Extend the Sync Listener to pass owner information to sentry server.

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

Re: Review Request 67134: SENTRY-2224: Support SHOW GRANT on HIVE_OBJECT

2018-06-19 Thread Arjun Mishra via Review Board
> On June 15, 2018, 9:02 p.m., kalyan kumar kalvagadda wrote: > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/SentryHiveAuthorizationTaskFactoryImpl.java > > Line 206 (original), 211 (patched) > >

Re: Review Request 67134: SENTRY-2224: Support SHOW GRANT on HIVE_OBJECT

2018-06-19 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67134/ --- (Updated June 19, 2018, 3:45 p.m.) Review request for sentry, kalyan kumar

Review Request 67649: SENTRY-2241: Extend the Sync Listener to pass owner information to sentry server.(server side)

2018-06-19 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67649/ --- Review request for sentry, Na Li and Sergio Pena. Bugs: SENTRY-2241

Re: Review Request 67134: SENTRY-2224: Support SHOW GRANT on HIVE_OBJECT

2018-06-19 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67134/ --- (Updated June 19, 2018, 4:08 p.m.) Review request for sentry, kalyan kumar

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

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

Re: Review Request 67650: SENTRY-2275: Grant and revoke owner privileges based on HMS updates(client-side)

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

Re: Review Request 67637: SENTRY-2265: Translate owner privilege in sentry binding for authorization

2018-06-19 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67637/#review205009 --- Ship it! Looks good. - kalyan kumar kalvagadda On June 18,

Re: Review Request 67650: SENTRY-2275: Grant and revoke owner privileges based on HMS updates(client-side)

2018-06-19 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67650/ --- (Updated June 19, 2018, 9:11 p.m.) Review request for sentry, Na Li and Sergio

Re: Review Request 67652: SENTRY-2271: Wrong message seen when db upgrade failure.

2018-06-19 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67652/ --- (Updated June 19, 2018, 9:52 p.m.) Review request for sentry, kalyan kumar

Review Request 67658: SENTRY-2269: Make SentryStore pluggable

2018-06-19 Thread Fahd Siddiqui
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67658/ --- Review request for sentry, kalyan kumar kalvagadda and Sergio Pena.