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

Rob Audenaerde edited comment on LUCENE-5476 at 3/11/14 8:10 AM:
-----------------------------------------------------------------

Thanks again for the good points. 

I currently have an {{amortizeFacetCounts}} that uses the {{IndexSearcher}} to 
retrieve a reader an a {{FacetsConfig}} to determine the {{Term}} for the 
{{docFreq}}. I'm not really sure how this will work for hierarchies though. 

Using {{totalHits}} as upper bound is not really necessary I think; as the 
sampling rate is determined by the total number of hits and the sample size; so 
reversing this can never yield numbers greater that {{totalHits}}. 

I alse removed the exact assert and switch to an {{atLeast}}. Will add a patch 
soon.




was (Author: robau):
Thanks again for the good points. 

I currently have an {amortizeFacetCounts} that uses the {IndexSearcher} to 
retrieve a reader an a {FacetsConfig} to determine the {Term} for the 
{docFreq}. I'm not really sure how this will work for hierarchies though. I 
will also add the {{totalHits}} as upper bound, great idea Gilad.

I alse removed the exact assert and switch to an atLeast. Will add a patch soon.



> Facet sampling
> --------------
>
>                 Key: LUCENE-5476
>                 URL: https://issues.apache.org/jira/browse/LUCENE-5476
>             Project: Lucene - Core
>          Issue Type: Improvement
>            Reporter: Rob Audenaerde
>         Attachments: LUCENE-5476.patch, LUCENE-5476.patch, LUCENE-5476.patch, 
> LUCENE-5476.patch, LUCENE-5476.patch, LUCENE-5476.patch, LUCENE-5476.patch, 
> SamplingComparison_SamplingFacetsCollector.java, SamplingFacetsCollector.java
>
>
> With LUCENE-5339 facet sampling disappeared. 
> When trying to display facet counts on large datasets (>10M documents) 
> counting facets is rather expensive, as all the hits are collected and 
> processed. 
> Sampling greatly reduced this and thus provided a nice speedup. Could it be 
> brought back?



--
This message was sent by Atlassian JIRA
(v6.2#6252)

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

Reply via email to