----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65768/ -----------------------------------------------------------
(Updated May 18, 2018, 9:23 p.m.) Review request for sentry, Alexander Kolbasov, kalyan kumar kalvagadda, and Sergio Pena. Summary (updated) ----------------- SENTRY-2144: Table Rename Cross Database should update permission correctly Repository: sentry Description ------- fix the bug that moves privilege to wrong table when DB name changes in alter table rename This fix only handles notification event processing when there is no full snapshot. The changes related to update to full snapshot is tracked in "SENTRY-2239 FullUpdateModifier does not handle alter table rename correctly" Diffs ----- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java cafe2b5 sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegeCleanupOnDrop.java 5fe6625 Diff: https://reviews.apache.org/r/65768/diff/5/ Testing ------- unit test succeeded Thanks, Na Li
