Re: Review Request 49329: GEODE-1607: ConcurrentModificationException could occur when iterating through hostedTXStates HashMap during closing cache

2016-06-29 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49329/#review140048 --- Ship it! Ship It! - Darrel Schneider On June 29, 2016, 1 p.m

Re: Review Request 49329: GEODE-1607: ConcurrentModificationException could occur when iterating through hostedTXStates HashMap during closing cache

2016-06-29 Thread Eric Shu
> On June 28, 2016, 9:36 p.m., Darrel Schneider wrote: > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/TXManagerImpl.java, > > line 589 > > > > > > Why do you release the sync BEFORE iterating the h

Re: Review Request 49329: GEODE-1607: ConcurrentModificationException could occur when iterating through hostedTXStates HashMap during closing cache

2016-06-29 Thread Eric Shu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49329/ --- (Updated June 29, 2016, 8 p.m.) Review request for geode, Darrel Schneider and

Re: Review Request 49329: GEODE-1607: ConcurrentModificationException could occur when iterating through hostedTXStates HashMap during closing cache

2016-06-28 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49329/#review139868 --- geode-core/src/main/java/com/gemstone/gemfire/internal/cache/TXMa

Review Request 49329: GEODE-1607: ConcurrentModificationException could occur when iterating through hostedTXStates HashMap during closing cache

2016-06-28 Thread Eric Shu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49329/ --- Review request for geode, Darrel Schneider and Swapnil Bawaskar. Bugs: GEODE-16