Re: Review Request 48757: GEODE-11: Cleaning up old IndexRepositoryImpls when buckets move

2016-06-24 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48757/#review139408 --- geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/inte

Re: Review Request 48757: GEODE-11: Cleaning up old IndexRepositoryImpls when buckets move

2016-06-23 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48757/#review139313 --- Fix it, then Ship it! Ship It! geode-lucene/src/main/java/com

Re: Review Request 48757: GEODE-11: Cleaning up old IndexRepositoryImpls when buckets move

2016-06-21 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48757/ --- (Updated June 21, 2016, 5:40 p.m.) Review request for geode, anilkumar gingade,

Re: Review Request 48757: GEODE-11: Cleaning up old IndexRepositoryImpls when buckets move

2016-06-15 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48757/#review137879 --- Fix it, then Ship it! Ship It! geode-lucene/src/main/java/com

Re: Review Request 48757: GEODE-11: Cleaning up old IndexRepositoryImpls when buckets move

2016-06-15 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48757/#review137854 --- geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/inte

Re: Review Request 48757: GEODE-11: Cleaning up old IndexRepositoryImpls when buckets move

2016-06-15 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48757/ --- (Updated June 15, 2016, 9:36 p.m.) Review request for geode, anilkumar gingade,

Re: Review Request 48757: GEODE-11: Cleaning up old IndexRepositoryImpls when buckets move

2016-06-15 Thread Dan Smith
> 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 > > > > > > I think the method spelling is

Review Request 48757: GEODE-11: Cleaning up old IndexRepositoryImpls when buckets move

2016-06-15 Thread Dan Smith
--- 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 xiaoj