[jira] [Resolved] (SOLR-12964) Use advanceExact instead of advance in a few remaining json facet use cases

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Resolved] (SOLR-12964) Use advanceExact instead of advance in a few remaining json facet use cases

JIRA jira@apache.org

     [ https://issues.apache.org/jira/browse/SOLR-12964?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

David Smiley resolved SOLR-12964.
---------------------------------
       Resolution: Fixed
    Fix Version/s: 7.6

> Use advanceExact instead of advance in a few remaining json facet use cases
> ---------------------------------------------------------------------------
>
>                 Key: SOLR-12964
>                 URL: https://issues.apache.org/jira/browse/SOLR-12964
>             Project: Solr
>          Issue Type: Improvement
>      Security Level: Public(Default Security Level. Issues are Public)
>          Components: Facet Module
>    Affects Versions: 7.5
>            Reporter: Tim Underwood
>            Assignee: David Smiley
>            Priority: Major
>             Fix For: 7.6
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> This updates 2 places in the JSON Facets code that uses the advance()/docID() pattern instead of the simpler advanceExact().  Most other usages in the faceting code already make use of advanceExact().
> The only remaining usage of advance() in org.apache.solr.search.facet is in:
>  * UniqueAgg.BaseNumericAcc.collect
>  * HLLAgg..BaseNumericAcc.collect
> The code for those of those looks very similar and probably makes sense to update but it would require changing the return type of the protected docIdSetIterator() method to return a DocValuesIterator in order to be able to call the advanceExact() method.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]