> On June 15, 2016, 9:28 p.m., Jason Huynh wrote: > > geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/repository/IndexRepositoryImpl.java, > > line 62 > > <https://reviews.apache.org/r/48757/diff/1/?file=1420398#file1420398line62> > > > > I think the method spelling is incorrect. Has one too many p's.... > > Not from this checkin but maybe we can fix that?
Sure thing, I'll fix that. - Dan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48757/#review137854 ----------------------------------------------------------- On June 15, 2016, 9:13 p.m., Dan Smith wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48757/ > ----------------------------------------------------------- > > (Updated June 15, 2016, 9:13 p.m.) > > > Review request for geode, anilkumar gingade, Jason Huynh, nabarun nag, and > xiaojian zhou. > > > Repository: geode > > > Description > ------- > > When a bucket is moved, we leave the IndexRepositoryImpl open. But even > after the bucket moves back, we just dereference the old > IndexRepositoryImpl without closing it. We should make sure we always > invoke close on the IndexRepositoryImpl to clean up any resources the > IndexWriter is using. > > > Diffs > ----- > > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/AbstractRegion.java > 95854ec2b47e82be946315ee65218fe504075b79 > geode-core/src/test/java/com/gemstone/gemfire/test/fake/Fakes.java > 323c281baaaf10bcf17c4b421b333de52f08dccd > > geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/PartitionedRepositoryManager.java > 25010b04cf10d6216b91a8de29b8c92ea2db34bf > > geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/repository/IndexRepository.java > fab2c2a5df17f836c29d983a41632469354d3955 > > geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/repository/IndexRepositoryImpl.java > 110f85acd27a7c958357074ee0d30dccdf763567 > > geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/PartitionedRepositoryManagerJUnitTest.java > ec56381bb54baa2de3921850afbb659c7ecf8fc8 > > geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/repository/IndexRepositoryImplJUnitTest.java > 7d3caf89b42a72dc0f8a9d934578e28cb4a95731 > > Diff: https://reviews.apache.org/r/48757/diff/ > > > Testing > ------- > > > Thanks, > > Dan Smith > >
