----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67237/ -----------------------------------------------------------
Review request for sentry, Alexander Kolbasov, Arjun Mishra, kalyan kumar kalvagadda, and Sergio Pena. Bugs: sentry-2239 https://issues.apache.org/jira/browse/sentry-2239 Repository: sentry Description ------- When HMS updates when sentry is taking full snapshot, sentry will apply those events event though some of them are already included in full snapshot. Update the code, so when the table rename cross database, the full snapshot is updated correctly Diffs ----- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/FullUpdateModifier.java c30d982 sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java 0322cc3 sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestFullUpdateModifier.java c6be80d sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegeCleanupOnDrop.java cbfdb94 Diff: https://reviews.apache.org/r/67237/diff/1/ Testing ------- TestDbPrivilegeCleanupOnDrop TestSentryStore TestFullUpdateModifier Thanks, Na Li