Re: Review Request 58187: GEODE-2732 after auto-reconnect a server is restarted on the default port of 40404

2017-04-06 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58187/#review171213 --- Ship it! Ship It! - Udo Kohlmeyer On April 4, 2017, 8:46 p.m

Re: Review Request 58187: GEODE-2732 after auto-reconnect a server is restarted on the default port of 40404

2017-04-05 Thread Bruce Schuchardt
> On April 5, 2017, 11:24 p.m., Galen O'Sullivan wrote: > > geode-core/src/main/java/org/apache/geode/internal/tcp/Connection.java > > Line 1288 (original) > > > > > > No more JRockit! I'm hoping we've officially dr

Re: Review Request 58187: GEODE-2732 after auto-reconnect a server is restarted on the default port of 40404

2017-04-05 Thread Galen O'Sullivan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58187/#review171087 --- Looks good. I left a few questions on the test, but the non-test c

Re: Review Request 58187: GEODE-2732 after auto-reconnect a server is restarted on the default port of 40404

2017-04-05 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58187/#review171159 --- Ship it! Ship It! - Hitesh Khamesra On April 4, 2017, 8:46 p

Review Request 58187: GEODE-2732 after auto-reconnect a server is restarted on the default port of 40404

2017-04-04 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58187/ --- Review request for geode, Galen O'Sullivan, Hitesh Khamesra, and Udo Kohlmeyer.