----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59897/ -----------------------------------------------------------
Review request for sentry and Alexander Kolbasov. Repository: sentry Description ------- SENTRY-1795 has solved the issue of holes in the relevant tables. Instead of using auto-incrementing property, we now add the number along with the transaction itself. That being said, the current jira is mostly solved and the only thing that needs to be taken care of is to make sure the ID's returned from the DB are in order to avoid NN jumping the transaction ID's. Diffs ----- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java 37eb0b25e10ef69057599277aa5941ca05d52290 Diff: https://reviews.apache.org/r/59897/diff/1/ Testing ------- Trivial fix Thanks, Vamsee Yarlagadda
