Re: Review Request 47245: GEODE-1327 java.util.ConcurrentModificationException while updating log message

2016-05-22 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47245/#review134303 --- Ship it! Ship It! - Udo Kohlmeyer On May 11, 2016, 5:55 p.m.

Re: Review Request 47245: GEODE-1327 java.util.ConcurrentModificationException while updating log message

2016-05-12 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47245/#review132951 --- Ship it! Ship It! - Bruce Schuchardt On May 11, 2016, 5:55 p

Re: Review Request 47245: GEODE-1327 java.util.ConcurrentModificationException while updating log message

2016-05-12 Thread Hitesh Khamesra
> On May 12, 2016, 3:02 p.m., Bruce Schuchardt wrote: > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/membership/GMSJoinLeave.java, > > line 2186 > > > > > > wouldn't it be sim

Re: Review Request 47245: GEODE-1327 java.util.ConcurrentModificationException while updating log message

2016-05-12 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47245/#review132924 --- geode-core/src/main/java/com/gemstone/gemfire/distributed/interna

Review Request 47245: GEODE-1327 java.util.ConcurrentModificationException while updating log message

2016-05-11 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47245/ --- Review request for geode, Bruce Schuchardt, Jianxia Chen, and Udo Kohlmeyer. Re