Skip to content
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

[BUG] Explore adoption for interval change in Boolean queries #17456

Open
expani opened this issue Feb 25, 2025 · 2 comments
Open

[BUG] Explore adoption for interval change in Boolean queries #17456

expani opened this issue Feb 25, 2025 · 2 comments
Labels
bug Something isn't working good first issue Good for newcomers Search:Performance v3.0.0 Issues and PRs related to version 3.0.0

Comments

@expani
Copy link
Contributor

expani commented Feb 25, 2025

Describe the bug

The interval size used by CancellableBulkScorer needs to be increased in accordance with Lucene as it moved from 2048 → 4096 recently here

This bug is to fix this change and also track other places where we have code untouched for a long time.

Related component

Search:Performance

To Reproduce

Nothing to reproduce.
Found this discrepancy during debugging for a different performance issue.

Expected behavior

Ensure we don't maintain duplicate constants and start discussion on some mechanism to not be caught like a deer in front of headlights when we do another Lucene upgrade.

Additional Details

No response

@getsaurabh02
Copy link
Member

@expani can you take a look? Or find someone who can help here?

@getsaurabh02 getsaurabh02 added good first issue Good for newcomers v3.0.0 Issues and PRs related to version 3.0.0 labels Feb 26, 2025
@tonicwave
Copy link

Hi!

Can I take this this and fix this issue, please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers Search:Performance v3.0.0 Issues and PRs related to version 3.0.0
Projects
Status: 🆕 New
Development

No branches or pull requests

3 participants