----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58593/#review172564 -----------------------------------------------------------
Ship it! Ship It! - Alexander Kolbasov On April 20, 2017, 8:56 p.m., Lei Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58593/ > ----------------------------------------------------------- > > (Updated April 20, 2017, 8:56 p.m.) > > > Review request for sentry, Alexander Kolbasov, Hao Hao, kalyan kumar > kalvagadda, and Na Li. > > > Bugs: SENTRY-1710 > https://issues.apache.org/jira/browse/SENTRY-1710 > > > Repository: sentry > > > Description > ------- > > As discussed in SENTRY-1643 and SENTRY-1706, MSentryPermChange and > MSentryPathChange tables require that CHANGE ID of such changes to be unique > and consecutive. SENTRY-1644 addressed this requirement in the application > code, that requires multiple trips for each change record insertion, while it > raises the possibility of collision on change ID when there are concurrently > updates. > > This patch change the ```key-cache-size``` to 1, as Lina suggested. > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryPathChange.java > 4b42ed02d7b6996ee9e460f346ebfefc44be9734 > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryPermChange.java > a97d10ad92739ed17e543372024b6afb652fe0be > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo > 81a4c6ecb271d8f04fe8caab1b52e8b4a2813ba1 > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/DeltaTransactionBlock.java > 8d3c88ba24eef76c6c07d06fb426e27b1d7d7f6b > > > Diff: https://reviews.apache.org/r/58593/diff/1/ > > > Testing > ------- > > mvn test -Dtest=TestSentryStore > > > Thanks, > > Lei Xu > >