----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69832/ -----------------------------------------------------------
(Updated Jan. 29, 2019, 6:47 p.m.) Review request for sentry, Haley Reeve, kalyan kumar kalvagadda, and Na Li. Changes ------- Fixed a bug where we try to clear an Immutable Set. Bugs: SENTRY-2488 https://issues.apache.org/jira/browse/SENTRY-2488 Repository: sentry Description ------- We are not consistent with behavior in SentryHiveMetaStoreHook (not used anymore) which would cache privileges when authorizing show databases or show tables command. This needs to be added back Diffs (updated) ----- sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/DefaultSentryValidator.java 9de47b338 sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/HiveAuthzBindingHookBase.java 63d8d1c09 sentry-provider/sentry-provider-cache/src/main/java/org/apache/sentry/provider/cache/SimplePrivilegeCache.java 0ad4616c0 Diff: https://reviews.apache.org/r/69832/diff/3/ Changes: https://reviews.apache.org/r/69832/diff/2-3/ Testing ------- Thanks, Arjun Mishra