-
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
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! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Co-authored-by: gesong.samuel <[email protected]>
Description
This try to implement
ConstantScoreScorer#nextDocsAndScores