----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66263/#review202373 -----------------------------------------------------------
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? - Sergio Pena 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 > >
