You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sentry.apache.org by shen guoquan <gu...@intel.com> on 2015/01/29 11:58:16 UTC

Review Request 30410: SENTRY:Add upgrade scripts for general privilege model

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30410/
-----------------------------------------------------------

Review request for sentry, Xiaomeng Huang, Colin Ma, Dapeng Sun, Lenni Kuff, and Prasad Mujumdar.


Repository: sentry


Description
-------

Update the database script for the generic authorization model


Diffs
-----

  sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.derby.sql PRE-CREATION 
  sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.mysql.sql PRE-CREATION 
  sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.oracle.sql PRE-CREATION 
  sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.postgres.sql PRE-CREATION 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-db2-1.5.0.sql 171adec 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-derby-1.5.0.sql 171adec 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-mysql-1.5.0.sql 01cd60d 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-oracle-1.5.0.sql 5ff0081 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-postgres-1.5.0.sql c451e33 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-db2-1.4.0-to-1.5.0.sql aa94ad1 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-derby-1.4.0-to-1.5.0.sql 2499bd7 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-mysql-1.4.0-to-1.5.0.sql 3fc90f3 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-oracle-1.4.0-to-1.5.0.sql ded813d 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-postgres-1.4.0-to-1.5.0.sql 2fbd2ca 

Diff: https://reviews.apache.org/r/30410/diff/


Testing
-------


Thanks,

shen guoquan


Re: Review Request 30410: SENTRY:Add upgrade scripts for general privilege model

Posted by shen guoquan <gu...@intel.com>.

> On 一月 30, 2015, 8:47 a.m., Colin Ma wrote:
> > sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-oracle-1.4.0-to-1.5.0.sql, line 6
> > <https://reviews.apache.org/r/30410/diff/2/?file=841388#file841388line6>
> >
> >     it's the script for oracle, still mysql?

I am so sorry for this stupid mistake. I will fix it .Thank colin.


> On 一月 30, 2015, 8:47 a.m., Colin Ma wrote:
> > sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-postgres-1.4.0-to-1.5.0.sql, line 6
> > <https://reviews.apache.org/r/30410/diff/2/?file=841389#file841389line6>
> >
> >     it's the script for postgres, still mysql?

I am so sorry for this stupid mistake. I will fix it .Thank colin.


- shen


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30410/#review70338
-----------------------------------------------------------


On 一月 30, 2015, 8:18 a.m., shen guoquan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/30410/
> -----------------------------------------------------------
> 
> (Updated 一月 30, 2015, 8:18 a.m.)
> 
> 
> Review request for sentry, Xiaomeng Huang, Colin Ma, Dapeng Sun, Lenni Kuff, and Prasad Mujumdar.
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> Update the database script for the generic authorization model
> 
> 
> Diffs
> -----
> 
>   sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.derby.sql PRE-CREATION 
>   sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.mysql.sql PRE-CREATION 
>   sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.oracle.sql PRE-CREATION 
>   sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.postgres.sql PRE-CREATION 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-db2-1.5.0.sql 171adec 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-derby-1.5.0.sql 171adec 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-mysql-1.5.0.sql 01cd60d 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-oracle-1.5.0.sql 5ff0081 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-postgres-1.5.0.sql c451e33 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-db2-1.4.0-to-1.5.0.sql aa94ad1 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-derby-1.4.0-to-1.5.0.sql 2499bd7 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-mysql-1.4.0-to-1.5.0.sql 3fc90f3 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-oracle-1.4.0-to-1.5.0.sql ded813d 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-postgres-1.4.0-to-1.5.0.sql 2fbd2ca 
> 
> Diff: https://reviews.apache.org/r/30410/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> shen guoquan
> 
>


Re: Review Request 30410: SENTRY:Add upgrade scripts for general privilege model

Posted by Colin Ma <ju...@intel.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30410/#review70338
-----------------------------------------------------------



sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-oracle-1.4.0-to-1.5.0.sql
<https://reviews.apache.org/r/30410/#comment115460>

    it's the script for oracle, still mysql?



sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-postgres-1.4.0-to-1.5.0.sql
<https://reviews.apache.org/r/30410/#comment115461>

    it's the script for postgres, still mysql?


- Colin Ma


On Jan. 30, 2015, 8:18 a.m., shen guoquan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/30410/
> -----------------------------------------------------------
> 
> (Updated Jan. 30, 2015, 8:18 a.m.)
> 
> 
> Review request for sentry, Xiaomeng Huang, Colin Ma, Dapeng Sun, Lenni Kuff, and Prasad Mujumdar.
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> Update the database script for the generic authorization model
> 
> 
> Diffs
> -----
> 
>   sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.derby.sql PRE-CREATION 
>   sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.mysql.sql PRE-CREATION 
>   sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.oracle.sql PRE-CREATION 
>   sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.postgres.sql PRE-CREATION 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-db2-1.5.0.sql 171adec 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-derby-1.5.0.sql 171adec 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-mysql-1.5.0.sql 01cd60d 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-oracle-1.5.0.sql 5ff0081 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-postgres-1.5.0.sql c451e33 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-db2-1.4.0-to-1.5.0.sql aa94ad1 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-derby-1.4.0-to-1.5.0.sql 2499bd7 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-mysql-1.4.0-to-1.5.0.sql 3fc90f3 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-oracle-1.4.0-to-1.5.0.sql ded813d 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-postgres-1.4.0-to-1.5.0.sql 2fbd2ca 
> 
> Diff: https://reviews.apache.org/r/30410/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> shen guoquan
> 
>


Re: Review Request 30410: SENTRY:Add upgrade scripts for general privilege model

Posted by Colin Ma <ju...@intel.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30410/#review74209
-----------------------------------------------------------

Ship it!


Ship It!

- Colin Ma


On Jan. 30, 2015, 8:59 a.m., shen guoquan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/30410/
> -----------------------------------------------------------
> 
> (Updated Jan. 30, 2015, 8:59 a.m.)
> 
> 
> Review request for sentry, Xiaomeng Huang, Colin Ma, Dapeng Sun, Lenni Kuff, and Prasad Mujumdar.
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> Update the database script for the generic authorization model
> 
> 
> Diffs
> -----
> 
>   sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.derby.sql PRE-CREATION 
>   sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.mysql.sql PRE-CREATION 
>   sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.oracle.sql PRE-CREATION 
>   sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.postgres.sql PRE-CREATION 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-db2-1.5.0.sql 171adec 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-derby-1.5.0.sql 171adec 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-mysql-1.5.0.sql 01cd60d 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-oracle-1.5.0.sql 5ff0081 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-postgres-1.5.0.sql c451e33 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-db2-1.4.0-to-1.5.0.sql aa94ad1 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-derby-1.4.0-to-1.5.0.sql 2499bd7 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-mysql-1.4.0-to-1.5.0.sql 3fc90f3 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-oracle-1.4.0-to-1.5.0.sql ded813d 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-postgres-1.4.0-to-1.5.0.sql 2fbd2ca 
> 
> Diff: https://reviews.apache.org/r/30410/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> shen guoquan
> 
>


Re: Review Request 30410: SENTRY:Add upgrade scripts for general privilege model

Posted by shen guoquan <gu...@intel.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30410/
-----------------------------------------------------------

(Updated 一月 30, 2015, 8:59 a.m.)


Review request for sentry, Xiaomeng Huang, Colin Ma, Dapeng Sun, Lenni Kuff, and Prasad Mujumdar.


Repository: sentry


Description
-------

Update the database script for the generic authorization model


Diffs (updated)
-----

  sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.derby.sql PRE-CREATION 
  sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.mysql.sql PRE-CREATION 
  sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.oracle.sql PRE-CREATION 
  sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.postgres.sql PRE-CREATION 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-db2-1.5.0.sql 171adec 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-derby-1.5.0.sql 171adec 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-mysql-1.5.0.sql 01cd60d 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-oracle-1.5.0.sql 5ff0081 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-postgres-1.5.0.sql c451e33 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-db2-1.4.0-to-1.5.0.sql aa94ad1 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-derby-1.4.0-to-1.5.0.sql 2499bd7 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-mysql-1.4.0-to-1.5.0.sql 3fc90f3 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-oracle-1.4.0-to-1.5.0.sql ded813d 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-postgres-1.4.0-to-1.5.0.sql 2fbd2ca 

Diff: https://reviews.apache.org/r/30410/diff/


Testing
-------


Thanks,

shen guoquan


Re: Review Request 30410: SENTRY:Add upgrade scripts for general privilege model

Posted by shen guoquan <gu...@intel.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30410/
-----------------------------------------------------------

(Updated 一月 30, 2015, 8:18 a.m.)


Review request for sentry, Xiaomeng Huang, Colin Ma, Dapeng Sun, Lenni Kuff, and Prasad Mujumdar.


Repository: sentry


Description
-------

Update the database script for the generic authorization model


Diffs (updated)
-----

  sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.derby.sql PRE-CREATION 
  sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.mysql.sql PRE-CREATION 
  sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.oracle.sql PRE-CREATION 
  sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.postgres.sql PRE-CREATION 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-db2-1.5.0.sql 171adec 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-derby-1.5.0.sql 171adec 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-mysql-1.5.0.sql 01cd60d 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-oracle-1.5.0.sql 5ff0081 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-postgres-1.5.0.sql c451e33 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-db2-1.4.0-to-1.5.0.sql aa94ad1 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-derby-1.4.0-to-1.5.0.sql 2499bd7 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-mysql-1.4.0-to-1.5.0.sql 3fc90f3 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-oracle-1.4.0-to-1.5.0.sql ded813d 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-postgres-1.4.0-to-1.5.0.sql 2fbd2ca 

Diff: https://reviews.apache.org/r/30410/diff/


Testing
-------


Thanks,

shen guoquan