> 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
> 
>

Reply via email to