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

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

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

2018-06-21 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 21, 2018, 9:28 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 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 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 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 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 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-19 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67648/#review205034 ---

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 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 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 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 ---