You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucene.apache.org by "Hoss Man (JIRA)" <ji...@apache.org> on 2018/07/23 16:32:00 UTC

[jira] [Commented] (SOLR-12567) Enhance JSON Facet aggregation syntax to support a JSON Object based "type:func" form that supports local params

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

Hoss Man commented on SOLR-12567:
---------------------------------

I went ahead and created SOLR-12581 as an initial sanity check that this new syntax/imple would actually be useful for some of optional aggregation features i'm anticipating.  apply that patch on top of this one to see a practical usecase.

> Enhance JSON Facet aggregation syntax to support a JSON Object based "type:func" form that supports local params
> ----------------------------------------------------------------------------------------------------------------
>
>                 Key: SOLR-12567
>                 URL: https://issues.apache.org/jira/browse/SOLR-12567
>             Project: Solr
>          Issue Type: Improvement
>      Security Level: Public(Default Security Level. Issues are Public) 
>            Reporter: Hoss Man
>            Assignee: Hoss Man
>            Priority: Major
>         Attachments: SOLR-12567.patch
>
>
> In SOLR-11709 I proposed adding an "expanded" JSON Object syntax for specifying aggregation stats, as a way to ultimately support applying domain changes to aggregations.
> Having since done more work with the implementation of aggregations and how they use SlotAcc(umulators) and how bucket sorting on aggregations work, I now realize that the goal of SOLR-11709 is pretty hard -- but the idea of this "expanded syntax" would still be useful as a way to specify Local Params to the ValueSourceParser used for the aggregation -- and would but stats on par with other {{"type"}} facets (syntactically).
> For things like {{relatedness(...)}} (or other user defined custom aggregations) it would allow for new optional configuration using "named" params w/o needing an unweildy function syntax (see [comment#16466664|https://issues.apache.org/jira/browse/SOLR-9480?focusedCommentId=16466664&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-16466664] in SOLR-9480).  But even for simple aggregations the expanded syntax may help folks with readability of expressions(w/o needing to resort to global params) ...
> Proposed syntax...
> {noformat}
>   "average_roi" : {
>     "type": "func",
>     "func": "avg(div($numer,$denom))",
>     "numer": "mul(popularity,rating)",
>     "denom": "mul(price,size)"
>   }
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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