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

Robert Muir commented on LUCENE-5476:
-------------------------------------

{quote}
Also, one thing that we did in the old impl is not use Math.random() or Random 
at all. Rather some crazy formula was used to create faster samples. I don't 
say we need to do that now, but maybe drop a TODO. At any rate, I don't think 
we should use Math.random(), but rather init a Random once and call 
nextDouble(). This will also allow to reuse a seed.
{quote}

xor-shift might be a good choice here.

> 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, 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.1.5#6160)

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

Reply via email to