[ https://issues.apache.org/jira/browse/LUCENE-4043?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13284417#comment-13284417 ]
Michael McCandless commented on LUCENE-4043: -------------------------------------------- Patch looks great! You don't need to use your own growFactor ... just call ArrayUtil.grow directly (it already oversizes under the hood for you). Maybe remove @throws IAE from createJoinQuery's javadocs? (But, still throw it... in case we add a new ScoreMode and forget to fix this code, in the future). Because today all ScoreMode enum values work... Fix omitted to emitted in the comment on top of "class MVInnerScorer". Probably javadocs should somewhere explain about the "first time doc is emitted it gets that score"? Maybe explain added RAM requirements when scores are tracked in the javadocs? Maybe rename TermsWithScoreCollector.MV.Avg.ordScores -> .scoreCounts (and .scores -> .scoreSums?). Can we put back the non-wildcard imports? > Add scoring support for query time join > --------------------------------------- > > Key: LUCENE-4043 > URL: https://issues.apache.org/jira/browse/LUCENE-4043 > Project: Lucene - Java > Issue Type: Improvement > Components: modules/join > Reporter: Martijn van Groningen > Attachments: LUCENE-4043.patch, LUCENE-4043.patch, LUCENE-4043.patch > > > Have similar scoring for query time joining just like the index time block > join (with the score mode). -- 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