----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61863/ -----------------------------------------------------------
(Updated April 5, 2018, 8:20 p.m.) Review request for sentry, kalyan kumar kalvagadda, Na Li, and Sergio Pena. Repository: sentry Description (updated) ------- Right now when we get privileges from sentry, we pass in a provider like set of groups. Then we create a MSentryGroup object for each group and then get roles using the .getRoles() method. This is bad since we only need the roleNames for the group and not the entire Role object. Instead running a SQL like query and just getting roleNames will drastically improve performance Diffs ----- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java d7acaea7c sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java 8f60b5801 Diff: https://reviews.apache.org/r/61863/diff/3/ Testing ------- Thanks, Arjun Mishra
