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

(Updated Aug. 25, 2017, 7:25 p.m.)


Review request for sentry, Alexander Kolbasov, Sergio Pena, and Vamsee 
Yarlagadda.


Repository: sentry


Description
-------

reduce field size `SERVER_NAME`,`DB_NAME`,`TABLE_NAME`,`COLUMN_NAME` in 
`SENTRY_DB_PRIVILEGE` from 4000 to 128, and remove `URI` from being part of the 
unique id


Diffs (updated)
-----

  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo
 734ea7f 


Diff: https://reviews.apache.org/r/61793/diff/2/

Changes: https://reviews.apache.org/r/61793/diff/1-2/


Testing
-------

run datanucleus schema tool. Without this change, it complains 
"com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: Specified key was 
too long; max key length is 767 bytes. 
 After those change, the datanucleus schema tool finishes successfully with 
command "mvn datanucleus:schema-create -X -e"


Thanks,

Na Li

Reply via email to