Re: Review Request 55280: GEODE-2142: Remove JSON from pulse

2017-01-09 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55280/#review160914 --- Ship it! Ship It! - Kirk Lund On Jan. 6, 2017, 9:09 p.m., An

Re: Review Request 55280: GEODE-2142: Remove JSON from pulse

2017-01-06 Thread Anthony Baker
> On Jan. 7, 2017, 12:27 a.m., Mark Bretl wrote: > > Will there be another changeset to remove the code from geode-json > > sub-project as well? Yes, we will need one. That's a more complicated change because we can't use a drop-in replacement (see GEODE-629). - Anthony --

Re: Review Request 55280: GEODE-2142: Remove JSON from pulse

2017-01-06 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55280/#review160795 --- Ship it! Ship It! - Jared Stewart On Jan. 6, 2017, 9:09 p.m.

Re: Review Request 55280: GEODE-2142: Remove JSON from pulse

2017-01-06 Thread Mark Bretl
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55280/#review160793 --- Ship it! Will there be another changeset to remove the code fro

Re: Review Request 55280: GEODE-2142: Remove JSON from pulse

2017-01-06 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55280/#review160766 --- Ship it! Ship It! - Jinmei Liao On Jan. 6, 2017, 9:09 p.m.,