> On May 4, 2018, 8:21 p.m., kalyan kumar kalvagadda wrote: > > sentry-provider/sentry-provider-db/src/main/resources/011-SENTRY-2154.derby.sql > > Lines 24-44 (patched) > > <https://reviews.apache.org/r/66263/diff/4/?file=2016931#file2016931line24> > > > > Lina, Is the the user privileges are implemented for both db model and > > generic model? > > Na Li wrote: > yes
I wasn't sure if it was implemeted for both the models. I was just conserned that we end up having some stale tables which are not used. - kalyan kumar ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66263/#review202475 ----------------------------------------------------------- On May 4, 2018, 7:25 p.m., Na Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66263/ > ----------------------------------------------------------- > > (Updated May 4, 2018, 7:25 p.m.) > > > Review request for sentry, Alexander Kolbasov, Arjun Mishra, kalyan kumar > kalvagadda, and Sergio Pena. > > > Bugs: sentry-2154 > https://issues.apache.org/jira/browse/sentry-2154 > > > 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/011-SENTRY-2154.derby.sql > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/resources/011-SENTRY-2154.mysql.sql > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/resources/011-SENTRY-2154.oracle.sql > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/resources/011-SENTRY-2154.postgres.sql > PRE-CREATION > 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 > > sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-derby-2.0.0-to-2.1.0.sql > 5640b76 > > sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-mysql-2.0.0-to-2.1.0.sql > 350bc48 > > sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-oracle-2.0.0-to-2.1.0.sql > 3afd788 > > sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-postgres-2.0.0-to-2.1.0.sql > 13df239 > > > Diff: https://reviews.apache.org/r/66263/diff/4/ > > > Testing > ------- > > Run the script and update script, and the tables are created and updated > successfully. > > > Thanks, > > Na Li > >
