-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61696/
-----------------------------------------------------------

Review request for sentry, kalyan kumar kalvagadda, Na Li, and Vamsee 
Yarlagadda.


Bugs: sentry-1803
    https://issues.apache.org/jira/browse/sentry-1803


Repository: sentry


Description
-------

This patch made changes on the SENTRY_PATH_CHANGES and 
SENTRY_HMS_NOTIFICATION_ID to allow duplicated IDs so that we handle HMS 
notifications with same event IDs correctly. 

It also adds a few test cases to check that duplicated events IDs are handled 
correctly.


Diffs
-----

  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo
 e8982f62b51d2db98ff280a42d01ad22333fd4ec 
  
sentry-provider/sentry-provider-db/src/main/resources/008-SENTRY-1569.derby.sql 
2a017ff069e7bd9822258b59a8bb9ea8f0049c44 
  
sentry-provider/sentry-provider-db/src/main/resources/008-SENTRY-1569.mysql.sql 
b587e40ac106071e742343423d9d7a52ebed45c7 
  
sentry-provider/sentry-provider-db/src/main/resources/008-SENTRY-1569.oracle.sql
 860f99269d294c66c136ab2e575065458e31992d 
  
sentry-provider/sentry-provider-db/src/main/resources/008-SENTRY-1569.postgres.sql
 c7c38e3b7201398287fcdadb605639f8b4f0b02a 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-db2-2.0.0.sql 
01be5098f3ba44624896c3668dad3dd6342ff1f4 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-derby-2.0.0.sql 
dca9e47bb2c7afab88e14da68f6f1425f687c117 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-mysql-2.0.0.sql 
0189724d9059639aa1ccce38c0793c9df1c87989 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-oracle-2.0.0.sql 
ecb76fc656a4fde4b5874df4b22660acd30b52bd 
  
sentry-provider/sentry-provider-db/src/main/resources/sentry-postgres-2.0.0.sql 
3e5d554cbf716ef72c341be7d76eb78982ba0303 
  
sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-db2-1.7.0-to-2.0.0.sql
 ac85968ebd2efbd7652ec45f8a2ddfa3398e7e48 
  
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java
 d35cafbd6189986d51221d867437532701813612 


Diff: https://reviews.apache.org/r/61696/diff/1/


Testing
-------

All tests passed.


Thanks,

Sergio Pena

Reply via email to