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

(Updated May 29, 2018, 6:29 p.m.)


Review request for sentry, Na Li and Sergio Pena.


Changes
-------

Addressed review comments.


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


Repository: sentry


Description
-------

Based in the notifications from HMS sentry should construct privilege 
(TSentryPrivilege) to grant/revoke privilege from the user.

 

This Jira is created to have API to construct TSentryPrivilege for a given 
authorizable.


Diffs (updated)
-----

  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyStoreProcessor.java
 816cfe182a9d42ba9b553ab10a149171b28d3727 
  
sentry-service/sentry-service-api/src/gen/thrift/gen-javabean/org/apache/sentry/api/service/thrift/SentryPolicyService.java
 fbbd725d31bccba516c266175bc8f993f4f3287f 
  
sentry-service/sentry-service-api/src/gen/thrift/gen-javabean/org/apache/sentry/api/service/thrift/TSentryHmsEventNotification.java
 PRE-CREATION 
  
sentry-service/sentry-service-api/src/gen/thrift/gen-javabean/org/apache/sentry/api/service/thrift/TSentryHmsEventNotificationResponse.java
 PRE-CREATION 
  
sentry-service/sentry-service-api/src/gen/thrift/gen-javabean/org/apache/sentry/api/service/thrift/TSentryObjectOwnerType.java
 PRE-CREATION 
  
sentry-service/sentry-service-api/src/main/resources/sentry_policy_service.thrift
 dcbd7e488bab46cce1b1369bb92c4549384ef38a 


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

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


Testing
-------

Added unit test to veirfy API.


Thanks,

kalyan kumar kalvagadda

Reply via email to