Re: Review Request 53920: SENTRY-1534: Oracle supports serializable instead of repeatable-read

2017-01-09 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53920/#review160957 --- Ship it! Ship It! - Hao Hao On Jan. 5, 2017, 9:04 a.m., Alex

Re: Review Request 53920: SENTRY-1534: Oracle supports serializable instead of repeatable-read

2017-01-05 Thread Alexander Kolbasov
> On Nov. 30, 2016, 11:55 p.m., Hao Hao wrote: > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java, > > line 167 > > > > > > Does "serializable" i

Re: Review Request 53920: SENTRY-1534: Oracle supports serializable instead of repeatable-read

2017-01-05 Thread Alexander Kolbasov
> On Jan. 5, 2017, 9:33 p.m., kalyan kumar kalvagadda wrote: > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java, > > line 154 > > > > > > Trying

Re: Review Request 53920: SENTRY-1534: Oracle supports serializable instead of repeatable-read

2017-01-05 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53920/#review160634 --- sentry-service/sentry-service-server/src/main/java/org/apache/sen

Re: Review Request 53920: SENTRY-1534: Oracle supports serializable instead of repeatable-read

2017-01-05 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53920/ --- (Updated Jan. 5, 2017, 9:04 a.m.) Review request for sentry, Colin Ma, Hao Hao,

Re: Review Request 53920: SENTRY-1534: Oracle supports serializable instead of repeatable-read

2016-11-30 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53920/#review157501 --- sentry-service/sentry-service-server/src/main/java/org/apache/sen

Re: Review Request 53920: SENTRY-1534: Oracle supports serializable instead of repeatable-read

2016-11-28 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53920/ --- (Updated Nov. 29, 2016, 6:29 a.m.) Review request for sentry, Colm O hEigeartai

Re: Review Request 53920: SENTRY-1534: Oracle supports serializable instead of repeatable-read

2016-11-28 Thread Hao Hao
> On Nov. 23, 2016, 1:38 a.m., Hao Hao wrote: > > sentry-service/sentry-service-common/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java, > > line 149 > > > > > > Could you explain a bit what is a

Re: Review Request 53920: SENTRY-1534: Oracle supports serializable instead of repeatable-read

2016-11-22 Thread Alexander Kolbasov
> On Nov. 23, 2016, 1:38 a.m., Hao Hao wrote: > > sentry-service/sentry-service-common/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java, > > line 149 > > > > > > Could you explain a bit what is a

Re: Review Request 53920: SENTRY-1534: Oracle supports serializable instead of repeatable-read

2016-11-22 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53920/#review156683 --- sentry-service/sentry-service-common/src/main/java/org/apache/sen

Review Request 53920: SENTRY-1534: Oracle supports serializable instead of repeatable-read

2016-11-18 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53920/ --- Review request for sentry, Colm O hEigeartaigh, Colin Ma, Hao Hao, Sravya Tirukk