[
https://issues.apache.org/jira/browse/SOLR-6351?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Hoss Man updated SOLR-6351:
---------------------------
Attachment: SOLR-6351.patch
small bits of progress on the patch review...
* PivotField
** fixed backcompt with constructor
** simplified write method & fixed NPE risk + indenting
* DistributedFacetPivotLargeTest
** refactored query + assertFieldStats into "doTestDeepPivotStats"
*** only called once, and wasn't general in anyway - only usable for checking
one specific query
** made the epsilon used in assertEqual(double) a tiny float (0.1E-7) instead
of a large float, so we don't overlook any bugs
* DistributedFacetPivotSmallTest
** simplified ComparablePivotField
*** it isn't used with stat assertions, and didn't check stats when it was
used, so it doesn't need to know about stats in it's constructor
** refactored query + assertFieldStats into "doTestDeepPivotStats"
*** only called once, and wasn't general in anyway - only usable for checking
one specific query
*** simplified params (no need to muck with setDistributedParams
** made the epsilon used in assertEqual(double) a tiny float (0.1E-7) instead
of a large float, so we don't overlook any bugs
* DistributedFacetPivotLongTailTest
** did the match to fill in the expected values of the remaining nocommits
** made the epsilon used in assertEqual(double) a tiny float (0.1E-7) instead
of a large float, so we don't overlook any bugs
> Let Stats Hang off of Pivots (via 'tag')
> ----------------------------------------
>
> Key: SOLR-6351
> URL: https://issues.apache.org/jira/browse/SOLR-6351
> Project: Solr
> Issue Type: Sub-task
> Reporter: Hoss Man
> Attachments: SOLR-6351.patch, SOLR-6351.patch, SOLR-6351.patch,
> SOLR-6351.patch, SOLR-6351.patch, SOLR-6351.patch, SOLR-6351.patch,
> SOLR-6351.patch, SOLR-6351.patch, SOLR-6351.patch
>
>
> he goal here is basically flip the notion of "stats.facet" on it's head, so
> that instead of asking the stats component to also do some faceting
> (something that's never worked well with the variety of field types and has
> never worked in distributed mode) we instead ask the PivotFacet code to
> compute some stats X for each leaf in a pivot. We'll do this with the
> existing {{stats.field}} params, but we'll leverage the {{tag}} local param
> of the {{stats.field}} instances to be able to associate which stats we want
> hanging off of which {{facet.pivot}}
> Example...
> {noformat}
> facet.pivot={!stats=s1}category,manufacturer
> stats.field={!key=avg_price tag=s1 mean=true}price
> stats.field={!tag=s1 min=true max=true}user_rating
> {noformat}
> ...with the request above, in addition to computing the min/max user_rating
> and mean price (labeled "avg_price") over the entire result set, the
> PivotFacet component will also include those stats for every node of the tree
> it builds up when generating a pivot of the fields "category,manufacturer"
--
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
