----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9481/ -----------------------------------------------------------
(Updated Feb. 17, 2013, 9:47 p.m.) Review request for pig, Ashutosh Chauhan, Dmitriy Ryaboy, Alan Gates, Jonathan Coveney, and Bill Graham. Changes ------- Please check out ny changes. I will work with Doug Daniels on fixing up the static class JsonSerializer in JsonStorage, which will effect both this and JsonStorage. Description ------- This patch is for JIRA, https://issues.apache.org/jira/browse/PIG-2641 PIG-2641 - which adds a ToJson builtin based on the serialization in JsonStorage. This addresses bug PIG-2641. https://issues.apache.org/jira/browse/PIG-2641 Diffs (updated) ----- src/org/apache/pig/builtin/JsonStorage.java 56086ff src/org/apache/pig/builtin/ToJson.java PRE-CREATION test/org/apache/pig/test/TestToJson.java PRE-CREATION test/org/apache/pig/test/data/jsonStorage1.tojson PRE-CREATION Diff: https://reviews.apache.org/r/9481/diff/ Testing ------- This works for me locally, and there is a working unit test. Thanks, Russell Jurney