Re: Review Request 59686: GEODE-2983: correctly handling --J option value that has ", " inside.

2017-06-02 Thread Emily Yeh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59686/#review176783 --- Ship it! Ship It! - Emily Yeh On May 31, 2017, 10:44 p.m., J

Re: Review Request 59686: GEODE-2983: correctly handling --J option value that has ", " inside.

2017-05-31 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59686/#review176544 --- Ship it! Ship It! - Jared Stewart On May 31, 2017, 10:44 p.m

Re: Review Request 59686: GEODE-2983: correctly handling --J option value that has ", " inside.

2017-05-31 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59686/ --- (Updated May 31, 2017, 10:44 p.m.) Review request for geode, Emily Yeh, Jared S

Re: Review Request 59686: GEODE-2983: correctly handling --J option value that has ", " inside.

2017-05-31 Thread Ken Howe
> On May 31, 2017, 10:18 p.m., Jared Stewart wrote: > > geode-core/src/main/java/org/apache/geode/management/internal/cli/GfshParser.java > > Lines 48 (patched) > > > > > > I worry that a user may at some point specif

Re: Review Request 59686: GEODE-2983: correctly handling --J option value that has ", " inside.

2017-05-31 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59686/#review176535 --- geode-core/src/main/java/org/apache/geode/management/internal/cli

Review Request 59686: GEODE-2983: correctly handling --J option value that has ", " inside.

2017-05-31 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59686/ --- Review request for geode, Emily Yeh, Jared Stewart, Ken Howe, Kirk Lund, and Pat