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

Review request for sentry and kalyan kumar kalvagadda.


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


Repository: sentry


Description
-------

before saving notification ID, check to make sure only save it if it is larger 
than the values of existing entries. We only track and use max(Notification 
ID). So no need to persist duplicated values.


Diffs
-----

  
sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
 980c8ad 
  
sentry-service/sentry-service-server/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java
 fd14963 


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


Testing
-------

add new unit tests, and they pass


Thanks,

Na Li

Reply via email to