lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Eks Dev (JIRA)" <>
Subject [jira] Commented: (LUCENE-1145) DisjunctionSumScorer small tweak
Date Fri, 08 Feb 2008 08:55:14 GMT


Eks Dev commented on LUCENE-1145:

test using Sun 1.4 jvm on the same hardware showed the same "a bit faster" behavior, so this
is in my opinion OK to be committed.   

> DisjunctionSumScorer small tweak
> --------------------------------
>                 Key: LUCENE-1145
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: Search
>         Environment: all
>            Reporter: Eks Dev
>            Priority: Trivial
>         Attachments: DisjunctionSumScorerOptimization.patch, DSSQueueSizeOptimization.patch,
> Move ScorerDocQueue initialization from next() and skipTo() methods to the Constructor.
Makes DisjunctionSumScorer a bit faster (less than 1% on my tests). 
> Downside (if this is one, I cannot judge) would be throwing IOException from DisjunctionSumScorer
constructors as we touch HardDisk there. I see no problem as this IOException does not propagate
too far (the only modification I made is in BooleanScorer2)
> if (scorerDocQueue == null) {
>       initScorerDocQueue();
> }
> Attached test is just quick & dirty rip of  TestScorerPerf from standard Lucene test
package. Not included as patch as I do not like it.
> All test pass, patch made on trunk revision 613923

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message