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

Review request for sentry and kalyan kumar kalvagadda.


Bugs: sentry-2265
    https://issues.apache.org/jira/browse/sentry-2265


Repository: sentry


Description
-------

Adds the OWNER action in the HiveActionFactory class which sets the OWNER 
action identical to the ALL action. 
This will ensure that the action checks behave like the ALL privilege.


Diffs
-----

  
sentry-core/sentry-core-model-db/src/main/java/org/apache/sentry/core/model/db/HiveActionFactory.java
 ad7e1c9f2b62c82e4006bcb1ec6bab440ddfac33 


Diff: https://reviews.apache.org/r/67637/diff/1/


Testing
-------

I cannot test this with unit tests because there are not grant/revoke to users 
yet. Unit tests for ownership will be added later once ownership is granted 
automatically in Sentry.
However, I did test it in a local cluster manually by modifying the database 
directly to add the owner privilege to a user. Then check that the user gets 
the correct authorization when the owner privilege is set and even show tables 
display only the tables of the owner.


Thanks,

Sergio Pena

Reply via email to