-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Implement ConstantScoreScorer#nextDocsAndScores
#14772
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Implement ConstantScoreScorer#nextDocsAndScores
#14772
Conversation
This PR does not have an entry in lucene/CHANGES.txt. Consider adding one. If the PR doesn't need a changelog entry, then add the skip-changelog label to it and you will stop receiving this reminder on future updates to the PR. |
Luceneutil benchmark result with
|
Interesting. I don't believe that being able to use |
@jpountz Thanks for your reply! (apologies for late reply). Actually I'm not very sure what's the root cause of this performance gain, My previous thought is that this change can co-operate with other query which implemnts the
|
Thanks for contribution, it's a nice speedup!
I wonder if we can benchmark the following implementation to confirm the source of speedup?
|
@gf2121 Thanks for your suggestion!
I'v done the benchmark according to your suggestion, here is the result under identical setup
The result shows a same speedup! |
Description
This try to implement
ConstantScoreScorer#nextDocsAndScores