----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55956/#review163079 -----------------------------------------------------------
Ship it! Looks good. I think the synchronization issues I mentioned in the last review are still present, but maybe it would be good to get this in and continue working on cleaning that up. - Dan Smith On Jan. 26, 2017, 1:15 a.m., Jason Huynh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55956/ > ----------------------------------------------------------- > > (Updated Jan. 26, 2017, 1:15 a.m.) > > > Review request for geode, Barry Oglesby, nabarun nag, Dan Smith, and xiaojian > zhou. > > > Repository: geode > > > Description > ------- > > Added afterSecondary callback to partition listener to allow cleaning up of > the index repo when the bucket losses primary > Added lock prior to creating the bucket indexes to prevent multiple index > writers from being available at a time > Changed single point of lucene index creation, no longer creating on the fly > > > Diffs > ----- > > > geode-core/src/main/java/org/apache/geode/cache/partition/PartitionListener.java > a534e50 > geode-core/src/main/java/org/apache/geode/internal/cache/BucketAdvisor.java > 7b79bfb > > geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/AbstractPartitionedRepositoryManager.java > aa29e1b > > geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/IndexRepositoryFactory.java > c73d64a > > geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LuceneEventListener.java > f2c7c8f > > geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/LucenePrimaryBucketListener.java > d17b5f2 > > geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/RawIndexRepositoryFactory.java > 2afccf9 > > geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/repository/IndexRepositoryImpl.java > 4e86eb5 > > geode-lucene/src/test/java/org/apache/geode/cache/lucene/internal/PartitionedRepositoryManagerJUnitTest.java > 960d794 > > geode-lucene/src/test/java/org/apache/geode/cache/lucene/internal/RawLuceneRepositoryManagerJUnitTest.java > 9201180 > > geode-lucene/src/test/java/org/apache/geode/cache/lucene/internal/distributed/DistributedScoringJUnitTest.java > 225f6ac > > Diff: https://reviews.apache.org/r/55956/diff/ > > > Testing > ------- > > > Thanks, > > Jason Huynh > >
