Re: Review Request 32374: HIVE-1575 get_json_object does not support JSON array at the root level

2015-03-25 Thread Jason Dere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32374/#review77767 --- ql/src/java/org/apache/hadoop/hive/ql/udf/UDFJson.java

Re: Review Request 32374: HIVE-1575 get_json_object does not support JSON array at the root level

2015-03-25 Thread Alexander Pivovarov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32374/ --- (Updated March 25, 2015, 8:29 p.m.) Review request for hive, Jason Dere and

Re: Review Request 32374: HIVE-1575 get_json_object does not support JSON array at the root level

2015-03-24 Thread Jason Dere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32374/#review77672 --- ql/src/java/org/apache/hadoop/hive/ql/udf/UDFJson.java

Re: Review Request 32374: HIVE-1575 get_json_object does not support JSON array at the root level

2015-03-24 Thread Alexander Pivovarov
On March 24, 2015, 11:30 p.m., Jason Dere wrote: ql/src/java/org/apache/hadoop/hive/ql/udf/UDFJson.java, line 136 https://reviews.apache.org/r/32374/diff/1/?file=902256#file902256line136 I don't know anything about JsonPath - so $[0], [0], $.[0] are all valid JsonPath? Should the

Re: Review Request 32374: HIVE-1575 get_json_object does not support JSON array at the root level

2015-03-24 Thread Alexander Pivovarov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32374/ --- (Updated March 25, 2015, 2:33 a.m.) Review request for hive, Jason Dere and

Review Request 32374: HIVE-1575 get_json_object does not support JSON array at the root level

2015-03-22 Thread Alexander Pivovarov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32374/ --- Review request for hive, Jason Dere and namit jain. Bugs: HIVE-1575