Re: Review Request 58996: GEODE-2876: Add logging to diagnose CI failure

2017-05-08 Thread Ken Howe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58996/#review174201 --- Ship it! Ship It! - Ken Howe On May 5, 2017, 11:15 p.m., Jar

Re: Review Request 58996: GEODE-2876: Add logging to diagnose CI failure

2017-05-08 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58996/#review174200 --- Ship it! Ship It! - Jinmei Liao On May 5, 2017, 11:15 p.m.,

Re: Review Request 58996: GEODE-2876: Add logging to diagnose CI failure

2017-05-08 Thread Patrick Rhomberg
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58996/#review174197 --- Ship it! Ship It! - Patrick Rhomberg On May 5, 2017, 11:15 p

Re: Review Request 58996: GEODE-2876: Add logging to diagnose CI failure

2017-05-05 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58996/ --- (Updated May 5, 2017, 11:15 p.m.) Review request for geode. Repository: geode

Re: Review Request 58996: GEODE-2876: Add logging to diagnose CI failure

2017-05-05 Thread Jinmei Liao
> On May 4, 2017, 5:29 p.m., Jinmei Liao wrote: > > geode-core/src/main/java/org/apache/geode/management/internal/cli/remote/CommandProcessor.java > > Line 124 (original), 125 (patched) > > > > > > we are not showing

Re: Review Request 58996: GEODE-2876: Add logging to diagnose CI failure

2017-05-05 Thread Jared Stewart
> On May 4, 2017, 5:29 p.m., Jinmei Liao wrote: > > geode-core/src/main/java/org/apache/geode/management/internal/cli/remote/CommandProcessor.java > > Line 124 (original), 125 (patched) > > > > > > we are not showing

Re: Review Request 58996: GEODE-2876: Add logging to diagnose CI failure

2017-05-04 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58996/#review173938 --- Ship it! Ship It! - Kirk Lund On May 4, 2017, 5:12 p.m., Jar

Re: Review Request 58996: GEODE-2876: Add logging to diagnose CI failure

2017-05-04 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58996/#review173930 --- geode-core/src/main/java/org/apache/geode/management/internal/cli