[
https://issues.apache.org/jira/browse/LUCENE-5144?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shai Erera updated LUCENE-5144:
-------------------------------
Attachment: LUCENE-5144.patch
Patch removes FacetRequest.createAggregator (NOTE: *not*
createFacetsAggregator) and replaces it by
StandardFacetsAccumulator.createAggregator(FacetRequest).
I also renamed SFA to OldFacetsAccumulator and moved it and all associated
classes under o.a.l.facet.old, in the intention of removing them one day.
> Nuke FacetRequest.createAggregator
> ----------------------------------
>
> Key: LUCENE-5144
> URL: https://issues.apache.org/jira/browse/LUCENE-5144
> Project: Lucene - Core
> Issue Type: Improvement
> Components: modules/facet
> Reporter: Shai Erera
> Assignee: Shai Erera
> Attachments: LUCENE-5144.patch
>
>
> Aggregator was replaced by FacetsAggregator. FacetRequest has
> createAggregator() which by default throws an UOE. It was left there until we
> migrate the aggregators to FacetsAggregator -- now all of our requests
> support FacetsAggregator.
> Aggregator is used only by StandardFacetsAccumulator, which too needs to
> vanish, at some point. But it currently it's the only one which handles
> sampling, complements aggregation and partitions.
> What I'd like to do is remove FacetRequest.createAggregator and in
> StandardFacetsAccumulator support only CountFacetRequest and
> SumScoreFacetRequest, which are the only ones that make sense for sampling
> and partitions. SumScore does not even support complements (which only work
> for counting).
> I'll also rename StandardFA to OldStandardFA. The plan is to eventually
> implement a SamplingAccumulator, PartitionsAccumulator/Aggregator and
> ComplementsAggregator, removing that class entirely. Until then ...
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
