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

(Updated Sept. 30, 2014, 4:41 p.m.)


Review request for sentry, Colin Ma, Lenni Kuff, and Sravya Tirukkovalur.


Changes
-------

Fixed testcase


Bugs: SENTRY-445
    https://issues.apache.org/jira/browse/SENTRY-445


Repository: sentry


Description
-------

The implied privilege for with grant is checking for action=  In other cases we 
allow action="ALL"
As a result with grant option for finer grain privileges doesn't work for 
clients that set action=ALL 


Diffs (updated)
-----

  
sentry-core/sentry-core-model-db/src/main/java/org/apache/sentry/core/model/db/AccessConstants.java
 26007d9 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryPrivilege.java
 0667cb5 
  
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryPrivilege.java
 91d3171 

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


Testing
-------

Added new test cases


Thanks,

Prasad Mujumdar

Reply via email to