Hi Yonik,

Your patch has corrected the thread thrashing problem on multi-cpu systems.
I've tested it with both 1.4.3 and 1.9. I haven't seen 100X performance
gain, but that's because I'm caching QueryFilters and Lucene is caching the
sort fields.

Thanks for the fast response!

btw, I had previously tried Chris's fix (replace synchronized method with
snapshot reference), but I was getting errors trying to fetch stored fields
from the Hits. I didn't chase it down, but the errors went away when I
reverted that specific patch.

Peter


On 10/12/05, Yonik Seeley <[EMAIL PROTECTED]> wrote:
>
> Here's the patch:
> http://issues.apache.org/jira/browse/LUCENE-454
>
> It resulted in quite a performance boost indeed!
>
> On 10/12/05, Yonik Seeley <[EMAIL PROTECTED]> wrote:
> >
> > Thanks for the trace Peter, and great catch!
> > It certainly does look like avoiding the construction of the docMap for
> a
> > MultiTermEnum will be a significant optimization.
> >
> >
> -Yonik
> Now hiring -- http://tinyurl.com/7m67g
>
>

Reply via email to