Re: Review Request 62195: SENTRY-1934: SQL Index name is too long for Oracle 11.2

2017-09-08 Thread Na Li
> On Sept. 8, 2017, 10:03 p.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo > > Line 323 (original), 323 (patched) > > > > > >

Re: Review Request 62195: SENTRY-1934: SQL Index name is too long for Oracle 11.2

2017-09-08 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62195/#review185023 --- Ship it! Ship It! - Alexander Kolbasov On Sept. 8, 2017, 9:5

Re: Review Request 62195: SENTRY-1934: SQL Index name is too long for Oracle 11.2

2017-09-08 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62195/#review185022 --- This is fine, but better would be to remove SENTRY from the name.

Review Request 62195: SENTRY-1934: SQL Index name is too long for Oracle 11.2

2017-09-08 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62195/ --- Review request for sentry, Alexander Kolbasov, Sergio Pena, and Vamsee Yarlagadd