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

(Updated June 4, 2018, 2:57 a.m.)


Review request for sentry, kalyan kumar kalvagadda and Sergio Pena.


Summary (updated)
-----------------

SENTRY-2244: Alter sentry role or user at granting privilege can avoid extra 
query to database


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


Repository: sentry


Description
-------

When granting privilege to a user or role, sentry does not need to get 
individual privilege with specific action from DB again after all its 
associated privileges are fetched from DB. We just just find the corresponding 
privilege from list to avoid extra query to DB


Diffs
-----

  
sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
 5932335 


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


Testing
-------

test cases in TestSentryStore succeeded


Thanks,

Na Li

Reply via email to