[ 
https://issues.apache.org/jira/browse/SOLR-402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12543411
 ] 

Ryan McKinley commented on SOLR-402:
------------------------------------

It looks like this patch is not really a JSON parser, it is more like a "raw" 
parser.  Rather then parsing the the response into a NamedList, it just puts 
the output into the named list:
{panel}
NamedList<Object> named = new NamedList<Object>();
named.add( "json", sb.toString() );
return named;
{panel}

Perhaps what you really want is RawResponseParser that just passes back the 
string?

If we do a JSON parser, I think it should actually parse the response into a 
NamedList -- QueryResponse only works if the NamedList is accuratly matched.  

Yonik has an interesting JSON parser at 
http://svn.apache.org/repos/asf/labs/noggit/ -- perhaps we should try that?

> JSON response support
> ---------------------
>
>                 Key: SOLR-402
>                 URL: https://issues.apache.org/jira/browse/SOLR-402
>             Project: Solr
>          Issue Type: New Feature
>          Components: clients - java
>    Affects Versions: 1.3
>         Environment: all
>            Reporter: AABones
>            Priority: Blocker
>             Fix For: 1.3
>
>         Attachments: jsonPatch.patch
>
>
> The Solrj java client was missing response support for JSON.  I added an 
> JSONResponseParser class and the necessary changes elsewhere to support it.  
> I'm attaching the patch file. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to