Re: Review Request 47323: GEODE-732 Unable to create PDXInstance from valid JSON using JSONFormatter

2016-05-13 Thread Jianxia Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47323/#review133143 --- There must be something missing that uses jsonListInsideList.txt.

Re: Review Request 47323: GEODE-732 Unable to create PDXInstance from valid JSON using JSONFormatter

2016-05-13 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47323/#review133141 --- geode-core/src/main/java/com/gemstone/gemfire/pdx/JSONFormatter.j

Re: Review Request 47323: GEODE-732 Unable to create PDXInstance from valid JSON using JSONFormatter

2016-05-13 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47323/#review133134 --- I see that JsonClientServerDUnitTest tests all of the documents in

Re: Review Request 47323: GEODE-732 Unable to create PDXInstance from valid JSON using JSONFormatter

2016-05-12 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47323/#review133013 --- I don't see the test class for this. Can you please attach it. -

Review Request 47323: GEODE-732 Unable to create PDXInstance from valid JSON using JSONFormatter

2016-05-12 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47323/ --- Review request for geode, Bruce Schuchardt, Jianxia Chen, and Udo Kohlmeyer. Re