----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66764/#review201960 -----------------------------------------------------------
Lina, would it be better to create another patch to fix the 2.1.0 versions? So we can have this patch to add the index only instead of several new files, what do you think? - Sergio Pena On April 23, 2018, 10:56 p.m., Na Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66764/ > ----------------------------------------------------------- > > (Updated April 23, 2018, 10:56 p.m.) > > > Review request for sentry, Alexander Kolbasov, Arjun Mishra, kalyan kumar > kalvagadda, and Sergio Pena. > > > Repository: sentry > > > Description > ------- > > add index for the foreign key AUTHZ_OBJ_ID on table AUTHZ_PATH > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStoreSchemaInfo.java > f1a5b10 > > sentry-provider/sentry-provider-db/src/main/resources/010-SENTRY-2210.derby.sql > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/resources/010-SENTRY-2210.mysql.sql > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/resources/010-SENTRY-2210.oracle.sql > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/resources/010-SENTRY-2210.postgres.sql > PRE-CREATION > sentry-provider/sentry-provider-db/src/main/resources/sentry-db2-2.1.0.sql > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/resources/sentry-derby-2.1.0.sql > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/resources/sentry-mysql-2.1.0.sql > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/resources/sentry-oracle-2.1.0.sql > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/resources/sentry-postgres-2.1.0.sql > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-db2-2.0.0-to-2.1.0.sql > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-derby-2.0.0-to-2.1.0.sql > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-mysql-2.0.0-to-2.1.0.sql > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-oracle-2.0.0-to-2.1.0.sql > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-postgres-2.0.0-to-2.1.0.sql > PRE-CREATION > sentry-provider/sentry-provider-db/src/main/resources/upgrade.order.db2 > 5ae9349 > sentry-provider/sentry-provider-db/src/main/resources/upgrade.order.derby > 770e1b5 > sentry-provider/sentry-provider-db/src/main/resources/upgrade.order.mysql > 770e1b5 > sentry-provider/sentry-provider-db/src/main/resources/upgrade.order.oracle > 770e1b5 > > sentry-provider/sentry-provider-db/src/main/resources/upgrade.order.postgres > 770e1b5 > > > Diff: https://reviews.apache.org/r/66764/diff/3/ > > > Testing > ------- > > run mysql script to verify it works on mysql > SentrySchemaTool upgrade/install tests > > > Thanks, > > Na Li > >
