----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70013/#review212980 -----------------------------------------------------------
Ship it! Ship It! - kalyan kumar kalvagadda On Feb. 19, 2019, 9:59 p.m., Na Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/70013/ > ----------------------------------------------------------- > > (Updated Feb. 19, 2019, 9:59 p.m.) > > > Review request for sentry, Arjun Mishra, Haley Reeve, and kalyan kumar > kalvagadda. > > > Bugs: sentry-2501 > https://issues.apache.org/jira/browse/sentry-2501 > > > Repository: sentry > > > Description > ------- > > The filter in SentryMetaStoreFilterHook does not cache sentry privileges. > Therefore, for each item in the list, sentry client has to get privileges > from sentry server. > > To improve performance, we need to add cache in SentryMetaStoreFilterHook > > > Diffs > ----- > > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/MetastoreAuthzBindingBase.java > cdb6de4 > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryMetaStoreFilterHook.java > 312c5db > > > Diff: https://reviews.apache.org/r/70013/diff/1/ > > > Testing > ------- > > existing HMS tests succeeded > > > Thanks, > > Na Li > >
