Re: Review Request 58325: GEODE-2737: Change Pulse UI tests to use random port for JMX connections

2017-04-12 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58325/#review171742 --- Ship it! Ship It! - Jinmei Liao On April 12, 2017, 4:02

Re: Review Request 58325: GEODE-2737: Change Pulse UI tests to use random port for JMX connections

2017-04-12 Thread Ken Howe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58325/ --- (Updated April 12, 2017, 4:02 p.m.) Review request for geode, Jinmei Liao,

Re: Review Request 58325: GEODE-2737: Change Pulse UI tests to use random port for JMX connections

2017-04-11 Thread Ken Howe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58325/ --- (Updated April 11, 2017, 9:28 p.m.) Review request for geode, Jinmei Liao,

Re: Review Request 58325: GEODE-2737: Change Pulse UI tests to use random port for JMX connections

2017-04-11 Thread Ken Howe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58325/ --- (Updated April 11, 2017, 5:18 p.m.) Review request for geode, Jinmei Liao,

Re: Review Request 58325: GEODE-2737: Change Pulse UI tests to use random port for JMX connections

2017-04-10 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58325/#review171502 --- Ship it! Ship It! - Jinmei Liao On April 10, 2017, 9:52

Re: Review Request 58325: GEODE-2737: Change Pulse UI tests to use random port for JMX connections

2017-04-10 Thread Jinmei Liao
> On April 10, 2017, 10:05 p.m., Kirk Lund wrote: > > geode-pulse/src/test/java/org/apache/geode/tools/pulse/tests/rules/ServerRule.java > > Lines 47 (patched) > > > > > > Is there no other way to configure this

Re: Review Request 58325: GEODE-2737: Change Pulse UI tests to use random port for JMX connections

2017-04-10 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58325/#review171499 --- Ship it! Ship It! - Jared Stewart On April 10, 2017, 9:52

Re: Review Request 58325: GEODE-2737: Change Pulse UI tests to use random port for JMX connections

2017-04-10 Thread Kirk Lund
> On April 10, 2017, 10:05 p.m., Kirk Lund wrote: > > geode-pulse/src/test/java/org/apache/geode/tools/pulse/tests/rules/ServerRule.java > > Lines 47 (patched) > > > > > > Is there no other way to configure this

Re: Review Request 58325: GEODE-2737: Change Pulse UI tests to use random port for JMX connections

2017-04-10 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58325/#review171495 ---

Review Request 58325: GEODE-2737: Change Pulse UI tests to use random port for JMX connections

2017-04-10 Thread Ken Howe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58325/ --- Review request for geode, Jinmei Liao, Jared Stewart, Kirk Lund, and Patrick