You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-dev@lucene.apache.org by "Ryan McKinley (JIRA)" <ji...@apache.org> on 2007/11/18 22:24:43 UTC

[jira] Commented: (SOLR-402) JSON response support

    [ 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.