----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61192/#review181721 -----------------------------------------------------------
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java Lines 3989-4030 (patched) <https://reviews.apache.org/r/61192/#comment257375> Try leveraging helper methods added to MSentryUtil and SentryUtils? sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java Lines 4028-4038 (original), 4093-4106 (patched) <https://reviews.apache.org/r/61192/#comment257372> I know you were trying to a simple revert but we added some helpful comments. Let's preserve them. - Vamsee Yarlagadda On July 28, 2017, 7:26 p.m., Na Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61192/ > ----------------------------------------------------------- > > (Updated July 28, 2017, 7:26 p.m.) > > > Review request for sentry, Alexander Kolbasov, Brian Towles, kalyan kumar > kalvagadda, Sergio Pena, and Vamsee Yarlagadda. > > > Repository: sentry > > > Description > ------- > > 1. Make perm changeID to be auto-increased for performance improvement > 2. revert change in sentry-1824 to synchronize the perm changes in SentryStore > 3. Return continuous delta list starting from requested changeID from > SentryStore even when the delta list has hole (for example, the list is > 1,2,3,5,6, return 1,2,3). If there is a hole at the front of the list, return > full snapshot. > Note: this only applies to perm changes. path changeID is manually increased, > so has no hole. > The changeID sent to NN in the full snapshot is in the same transaction of > getting the full snapshot. > > > Diffs > ----- > > > sentry-hdfs/sentry-hdfs-service/src/test/java/org/apache/sentry/hdfs/TestSentryHDFSServiceProcessor.java > f2368b7 > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryPathChange.java > 58878e5 > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryPermChange.java > e29e780 > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo > e8982f6 > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/DeltaTransactionBlock.java > 4fe62bf > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java > 670bc5e > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/model/TestMSentryUtil.java > eb39b0d > > > Diff: https://reviews.apache.org/r/61192/diff/1/ > > > Testing > ------- > > Tested by Vamsee in his cluster > > > Thanks, > > Na Li > >
