[ https://issues.apache.org/jira/browse/LUCENE-2686?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13260085#comment-13260085 ]
Michael McCandless commented on LUCENE-2686: -------------------------------------------- LiLi can you post the test case you created? Mikhail, can you post a patch with your ShouldQuery? I don't understand why you needed to modify CSQ.score to break the infinite loop... > DisjunctionSumScorer should not call .score on sub scorers until consumer > calls .score > -------------------------------------------------------------------------------------- > > Key: LUCENE-2686 > URL: https://issues.apache.org/jira/browse/LUCENE-2686 > Project: Lucene - Java > Issue Type: Bug > Components: core/search > Reporter: Michael McCandless > Assignee: Michael McCandless > Fix For: 4.0 > > Attachments: LUCENE-2686.patch, LUCENE-2686.patch, > Test2LUCENE2590.java > > > Spinoff from java-user thread "question about Scorer.freq()" from Koji... > BooleanScorer2 uses DisjunctionSumScorer to score only-SHOULD-clause boolean > queries. > But, this scorer does too much work for collectors that never call .score, > because it scores while it's matching. It should only call .score on the > subs when the caller calls its .score. > This also has the side effect of messing up advanced collectors that gather > the freq() of the subs (using LUCENE-2590). -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org