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

2018-06-12 Thread Na Li via Review Board
--- 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., Sergi

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

2018-06-12 Thread Sergio Pena via Review Board
--- 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. Bugs:

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

2018-06-12 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67527/#review204604 --- sentry-service/sentry-service-api/src/main/java/org/apache/sentry

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

2018-06-12 Thread Na Li via Review Board
> 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

2018-06-12 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67527/#review204602 --- sentry-service/sentry-service-api/src/main/java/org/apache/sentry

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

2018-06-11 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67527/ --- Review request for sentry and Na Li. Bugs: SENTRY-2262 https://issues.apach