----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63424/#review191753 -----------------------------------------------------------
Ship it! Ship It! - Sergio Pena On Nov. 22, 2017, 6:29 p.m., Na Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63424/ > ----------------------------------------------------------- > > (Updated Nov. 22, 2017, 6:29 p.m.) > > > Review request for sentry, Arjun Mishra, kalyan kumar kalvagadda, and Sergio > Pena. > > > Repository: sentry > > > Description > ------- > > AUTHZ_OBJ_NAME has (384) chars. It is defieved from DB_NAME and TBL_NAME from > hive. the constrain AUTHZOBJNAME will have error "Specified key was too long; > max key length is 767 bytes" for mysql. > > Since this index does not have much benefit, the solution is to drop the > index includes this field, so it works for mysql regardless the char set of > its table or DB. > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo > 2f2aabb > > sentry-provider/sentry-provider-db/src/main/resources/007-SENTRY-1365.derby.sql > 13708ba > > sentry-provider/sentry-provider-db/src/main/resources/007-SENTRY-1365.mysql.sql > a7db1b7 > > sentry-provider/sentry-provider-db/src/main/resources/007-SENTRY-1365.oracle.sql > ebb981a > > sentry-provider/sentry-provider-db/src/main/resources/007-SENTRY-1365.postgres.sql > e0b560c > sentry-provider/sentry-provider-db/src/main/resources/sentry-db2-2.0.0.sql > 2c1268a > > sentry-provider/sentry-provider-db/src/main/resources/sentry-derby-2.0.0.sql > 1c51049 > > sentry-provider/sentry-provider-db/src/main/resources/sentry-mysql-2.0.0.sql > 9f61154 > > sentry-provider/sentry-provider-db/src/main/resources/sentry-oracle-2.0.0.sql > 58241e5 > > sentry-provider/sentry-provider-db/src/main/resources/sentry-postgres-2.0.0.sql > 5d52136 > > sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-db2-1.8.0-to-2.0.0.sql > 5e8be9c > > > Diff: https://reviews.apache.org/r/63424/diff/4/ > > > Testing > ------- > > tested on system > > > Thanks, > > Na Li > >