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

Tomás Fernández Löbbe commented on SOLR-7214:
---------------------------------------------

bq. let's simply have a layer that can parse parameters into a JSON format that 
will be provided to it
When I read this Jira I thought this was the idea, it looks like it's much more 
than just the JSON API. I do think this is a great feature, but I agree that 
duplicating so much code (and current effort tracked in SOLR-6348) is not good.
bq.  I would prefer we resolve the user experience before we commit and release 
this.
+1, Once this gets released it's going to be much more difficult to refactor 
and do changes that could break backward compatibility or stability.

A random/unrelated thought, I think for patches like these (like 10k lines, 
including moved classes!) would be great candidates to use the review tool, it 
gets very difficult to review otherwise.
Also, it seems like the description of the FacetModule is "Heliosearch 
Faceting", is this intended?

> JSON Facet API
> --------------
>
>                 Key: SOLR-7214
>                 URL: https://issues.apache.org/jira/browse/SOLR-7214
>             Project: Solr
>          Issue Type: New Feature
>            Reporter: Yonik Seeley
>         Attachments: SOLR-7214.patch
>
>
> Overview is here: http://yonik.com/json-facet-api/
> The structured nature of nested sub-facets are more naturally expressed in a 
> nested structure like JSON rather than the flat structure that normal query 
> parameters provide.
> Goals:
> - First class JSON support
> - Easier programmatic construction of complex nested facet commands
> - Support a much more canonical response format that is easier for clients to 
> parse
> - First class analytics support
> - Support a cleaner way to do distributed faceting
> - Support better integration with other search features



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to