Re: Review Request 67555: SENTRY-2267: Listing user privileges fails because roleName field is required on Thrift

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67555/#review204636 --- Ship it! Ship It! - Na Li On June 12, 2018, 6:46 p.m.,

Re: Review Request 67539: SENTRY-2264: It is possible to elevate privileges from DROP using alter table rename

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67539/ --- (Updated June 12, 2018, 8:16 p.m.) Review request for sentry, kalyan kumar

Re: Review Request 67555: SENTRY-2267: Listing user privileges fails because roleName field is required on Thrift

> On June 12, 2018, 6:58 p.m., Arjun Mishra wrote: > > sentry-service/sentry-service-api/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyServiceClientDefaultImpl.java > > Lines 270 (patched) > > > > > >

Re: Review Request 67555: SENTRY-2267: Listing user privileges fails because roleName field is required on Thrift

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67555/#review204631 --- Ship it! Ship It! - Arjun Mishra On June 12, 2018, 6:46

Re: Review Request 67555: SENTRY-2267: Listing user privileges fails because roleName field is required on Thrift

> On June 12, 2018, 6:58 p.m., Arjun Mishra wrote: > > sentry-service/sentry-service-api/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyServiceClientDefaultImpl.java > > Lines 270 (patched) > > > > > >

Re: Review Request 67555: SENTRY-2267: Listing user privileges fails because roleName field is required on Thrift

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67555/#review204626 ---

Review Request 67555: SENTRY-2267: Listing user privileges fails because roleName field is required on Thrift

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67555/ --- Review request for sentry, Arjun Mishra, kalyan kumar kalvagadda, and Na Li.

Re: Review Request 67527: SENTRY-2262: Sentry client is not compatible when connecting to Sentry 2.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67527/#review204622 --- Ship it! Ship It! - Na Li On June 12, 2018, 4:27 p.m.,

Re: Review Request 67539: SENTRY-2264: It is possible to elevate privileges from DROP using alter table rename

> On June 12, 2018, 4:27 p.m., kalyan kumar kalvagadda wrote: > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/HiveAuthzPrivilegesMap.java > > Line 110 (original), 110-111 (patched) > >

Re: Review Request 67527: SENTRY-2262: Sentry client is not compatible when connecting to Sentry 2.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67527/ --- (Updated June 12, 2018, 4:27 p.m.) Review request for sentry and Na Li.

Re: Review Request 67539: SENTRY-2264: It is possible to elevate privileges from DROP using alter table rename

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67539/#review204605 ---

Re: Requesting contributor permission for Sentry project

Thanks ! I had the wrong address for the sentry dev group. Corrected and resend the email. On Tue, Jun 12, 2018 at 10:41 AM Fahd Siddiqui wrote: > Hi all, > > I would like to request contributor permission to the Sentry project. My > user name is "fahdsiddiqui". Let me know if there is anything

Requesting contributor permission for Sentry project

Hi all, I would like to request contributor permission to the Sentry project. My user name is "fahdsiddiqui". Let me know if there is anything else required of me. Thanks, Fahd

Re: Review Request 67527: SENTRY-2262: Sentry client is not compatible when connecting to Sentry 2.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67527/#review204604 ---

Re: Review Request 67527: SENTRY-2262: Sentry client is not compatible when connecting to Sentry 2.0

> On June 12, 2018, 4:05 p.m., Na Li wrote: > > sentry-service/sentry-service-api/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyServiceClientDefaultImpl.java > > Line 236 (original), 236 (patched) > > >

Re: Review Request 67527: SENTRY-2262: Sentry client is not compatible when connecting to Sentry 2.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67527/#review204602 ---

Re: Review Request 67528: SENTRY-2056: Display test-patch.py output on the standard console to see progress on Jenkins

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67528/#review204600 --- Ship it! Ship It! - Sergio Pena On June 11, 2018, 4:18