----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48757/ -----------------------------------------------------------
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
