-----------------------------------------------------------
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

Reply via email to