> On April 10, 2017, 11:37 a.m., Hao Hao wrote: > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java > > Lines 2581 (patched) > > <https://reviews.apache.org/r/58281/diff/1/?file=1686603#file1686603line2581> > > > > Can you add a comment for what this test is about? Also a link to the > > sentry jira number. Thanks!
Addressed in the updated patch. - Lei ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58281/#review171458 ----------------------------------------------------------- On April 7, 2017, 5:38 p.m., Lei Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58281/ > ----------------------------------------------------------- > > (Updated April 7, 2017, 5:38 p.m.) > > > Review request for sentry. > > > Bugs: SENTRY-1643 > https://issues.apache.org/jira/browse/SENTRY-1643 > > > Repository: sentry > > > Description > ------- > > When it relies on the SQL auto increment primary key as ChangeID, it can not > guarentee the consectivity of the IDs, because while each transaction claims > new ID from the table counter, the concurrent transactions which were not > success would not return the claimed changeIDs to the pool, thus it can not > guarateen the consectivity of change IDs. > > This patch changes to use application logic to force the consectivity of IDs. > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryPathChange.java > a0d34459d7b2f70e863ef6e078401df81381c91b > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryPermChange.java > 476fbcb2ad26de23757842111beb12b154e1562b > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/DeltaTransactionBlock.java > f590a5296c047e1acedd39a4f2e4f1de98008d32 > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java > 802b9c6cbf8e9ad015e37037b809b58c956de746 > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java > aaa0b9fd30bb68fded67f885af4f77bc71398e77 > > > Diff: https://reviews.apache.org/r/58281/diff/1/ > > > Testing > ------- > > Add a new test to conurrently insert changes. > > mvn test -Dtest=TestSentryStore. > > > Thanks, > > Lei Xu > >
