gsmiller commented on PR #12055: URL: https://github.com/apache/lucene/pull/12055#issuecomment-1442363428
Updated this PR in the following ways: 1. I've removed the changes to `TermInSetQuery` with the plan being to have `TermInSetQuery` extend from `MultiTermQuery` to get these benefits. 2. I've added the proposed logic as a _new_ `RewriteMethod` and left the existing ones unchanged. 3. I've made the new `RewriteMethod` ("blended") the default for non-scoring cases. Users could override this behavior with the "old" constant scoring rewrite method if necessary. 4. I've added/updated tests. I believe this addresses all the previous feedback and is ready for another look. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org