-----------------------------------------------------------
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