You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sentry.apache.org by Dapeng Sun <da...@intel.com> on 2015/08/31 10:25:59 UTC

Review Request 37940: SENTRY-504: Sentry Hive authorizer interfaces for authorization V2

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

Review request for sentry, Colin Ma, shen guoquan, and Sravya Tirukkovalur.


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


Repository: sentry


Description
-------

This jira contains three interfaces: **SentryHiveAuthorizer**,**SentryHiveAccessController** and **SentryHiveAuthorizationValidator**.

**SentryHiveAuthorizer** is the implementation of **HiveAuthorizer** in Sentry. 

**SentryHiveAccessController** is target to handle the operation likes grant/revoke.

**SentryHiveAuthorizationValidator** is target to check user's permission when do authorization and do metadata protection.


Diffs
-----

  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAccessController.java PRE-CREATION 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizationValidator.java PRE-CREATION 
  sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizer.java PRE-CREATION 

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


Testing
-------


Thanks,

Dapeng Sun


Re: Review Request 37940: SENTRY-504: Sentry Hive authorizer interfaces for authorization V2

Posted by Dapeng Sun <da...@intel.com>.

> On 九月 1, 2015, 9:38 a.m., Colin Ma wrote:
> > There is no pom.xml for this new project?

The pom.xml already committed to branch now. **SENTRY-765**  https://git1-us-west.apache.org/repos/asf?p=incubator-sentry.git;a=commit;h=03ad8bd1ec8a5b523b33de971ac031a859096a3f


- Dapeng


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


On 八月 31, 2015, 4:25 p.m., Dapeng Sun wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37940/
> -----------------------------------------------------------
> 
> (Updated 八月 31, 2015, 4:25 p.m.)
> 
> 
> Review request for sentry, Colin Ma, shen guoquan, and Sravya Tirukkovalur.
> 
> 
> Bugs: SENTRY-504
>     https://issues.apache.org/jira/browse/SENTRY-504
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> This jira contains three interfaces: **SentryHiveAuthorizer**,**SentryHiveAccessController** and **SentryHiveAuthorizationValidator**.
> 
> **SentryHiveAuthorizer** is the implementation of **HiveAuthorizer** in Sentry. 
> 
> **SentryHiveAccessController** is target to handle the operation likes grant/revoke.
> 
> **SentryHiveAuthorizationValidator** is target to check user's permission when do authorization and do metadata protection.
> 
> 
> Diffs
> -----
> 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAccessController.java PRE-CREATION 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizationValidator.java PRE-CREATION 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizer.java PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/37940/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Dapeng Sun
> 
>


Re: Review Request 37940: SENTRY-504: Sentry Hive authorizer interfaces for authorization V2

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


There is no pom.xml for this new project?

- Colin Ma


On Aug. 31, 2015, 8:25 a.m., Dapeng Sun wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37940/
> -----------------------------------------------------------
> 
> (Updated Aug. 31, 2015, 8:25 a.m.)
> 
> 
> Review request for sentry, Colin Ma, shen guoquan, and Sravya Tirukkovalur.
> 
> 
> Bugs: SENTRY-504
>     https://issues.apache.org/jira/browse/SENTRY-504
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> This jira contains three interfaces: **SentryHiveAuthorizer**,**SentryHiveAccessController** and **SentryHiveAuthorizationValidator**.
> 
> **SentryHiveAuthorizer** is the implementation of **HiveAuthorizer** in Sentry. 
> 
> **SentryHiveAccessController** is target to handle the operation likes grant/revoke.
> 
> **SentryHiveAuthorizationValidator** is target to check user's permission when do authorization and do metadata protection.
> 
> 
> Diffs
> -----
> 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAccessController.java PRE-CREATION 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizationValidator.java PRE-CREATION 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizer.java PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/37940/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Dapeng Sun
> 
>


Re: Review Request 37940: SENTRY-504: Sentry Hive authorizer interfaces for authorization V2

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

Ship it!


Ship It!

- Colin Ma


On Aug. 31, 2015, 8:25 a.m., Dapeng Sun wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37940/
> -----------------------------------------------------------
> 
> (Updated Aug. 31, 2015, 8:25 a.m.)
> 
> 
> Review request for sentry, Colin Ma, shen guoquan, and Sravya Tirukkovalur.
> 
> 
> Bugs: SENTRY-504
>     https://issues.apache.org/jira/browse/SENTRY-504
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> This jira contains three interfaces: **SentryHiveAuthorizer**,**SentryHiveAccessController** and **SentryHiveAuthorizationValidator**.
> 
> **SentryHiveAuthorizer** is the implementation of **HiveAuthorizer** in Sentry. 
> 
> **SentryHiveAccessController** is target to handle the operation likes grant/revoke.
> 
> **SentryHiveAuthorizationValidator** is target to check user's permission when do authorization and do metadata protection.
> 
> 
> Diffs
> -----
> 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAccessController.java PRE-CREATION 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizationValidator.java PRE-CREATION 
>   sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/SentryHiveAuthorizer.java PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/37940/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Dapeng Sun
> 
>