> On May 3, 2018, 8:30 p.m., Sergio Pena wrote: > > Lina, > > > > What about the rest of the sentry-upgrade-*-2.0.0-to-2.1.0.sql files? > > Such as sentry-upgrade-postgres, sentry-upgrade-oracle, > > sentry-upgrade-mysql and sentry-upgrade-derby?
The reason db2 has upgrade file sentry-upgrade-db2-2.0.0-to-2.1.0.sql is because it contains the new tables and indexes. Note there is no 010-SENTRY-2210.XXX.sql for db2. Such info for other DB types is in 010-SENTRY-2210.XXX.sql. And upgrade file for those DB types just contain running this 010-SENTRY-2210.XXX.sql. For example sentry-upgrade-mysql-2.0.0-to-2.1.0.sql SELECT 'Upgrading Sentry store schema from 2.0.0 to 2.1.0' AS ' '; SOURCE 010-SENTRY-2210.mysql.sql; UPDATE SENTRY_VERSION SET SCHEMA_VERSION='2.1.0', VERSION_COMMENT='Sentry release version 2.1.0' WHERE VER_ID=1; SELECT 'Finish upgrading Sentry store schema from 2.0.0 to 2.1.0' AS ' '; - Na ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66263/#review202373 ----------------------------------------------------------- On May 3, 2018, 3:49 a.m., Na Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66263/ > ----------------------------------------------------------- > > (Updated May 3, 2018, 3:49 a.m.) > > > Review request for sentry, Alexander Kolbasov, Arjun Mishra, kalyan kumar > kalvagadda, and Sergio Pena. > > > Repository: sentry > > > Description > ------- > > update mysql related scripts to > 1) dd new DB table SENTRY_USER_DB_PRIVILEGE_MAP and > SENTRY_USER_GM_PRIVILEGE_MAP to support grant user to privileges > 2) a flag is added in privilege table to indicate the privilege is created > by user, or created by sentry implicitly. I will make change in application > code so User can view the implicit privileges, but cannot change it directly > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/main/resources/010-SENTRY-2210.derby.sql > 5c55090 > > sentry-provider/sentry-provider-db/src/main/resources/010-SENTRY-2210.mysql.sql > 48c929e > > sentry-provider/sentry-provider-db/src/main/resources/010-SENTRY-2210.oracle.sql > 3fa60bb > > sentry-provider/sentry-provider-db/src/main/resources/010-SENTRY-2210.postgres.sql > cdc2ff7 > sentry-provider/sentry-provider-db/src/main/resources/sentry-db2-2.1.0.sql > 5e36ee3 > > sentry-provider/sentry-provider-db/src/main/resources/sentry-derby-2.1.0.sql > 374d59e > > sentry-provider/sentry-provider-db/src/main/resources/sentry-mysql-2.1.0.sql > 8876872 > > sentry-provider/sentry-provider-db/src/main/resources/sentry-oracle-2.1.0.sql > 46d3fc8 > > sentry-provider/sentry-provider-db/src/main/resources/sentry-postgres-2.1.0.sql > c28dd9f > > sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-db2-2.0.0-to-2.1.0.sql > 3e0373d > > > Diff: https://reviews.apache.org/r/66263/diff/2/ > > > Testing > ------- > > Run the script and update script, and the tables are created and updated > successfully. > > > Thanks, > > Na Li > >