----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69832/ -----------------------------------------------------------
(Updated Jan. 24, 2019, 9:35 p.m.) Review request for sentry, Haley Reeve, kalyan kumar kalvagadda, and Na Li. Changes ------- Post feebdack 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 Diff: https://reviews.apache.org/r/69832/diff/2/ Changes: https://reviews.apache.org/r/69832/diff/1-2/ Testing ------- Thanks, Arjun Mishra