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

Remove temp XIntervals class #119937

Open
wants to merge 1 commit into
base: lucene_snapshot_10_1
Choose a base branch
from

Conversation

mayya-sharipova
Copy link
Contributor

PR #114234 introduced XIntervals as a temp solution while waiting for
a fix in Lucene 10.1. With Lucene 10.1 we don't need this class anymore.

Related to #113554

PR elastic#114234 introduced XIntervals as a temp solution while waiting for
a fix in Lucene 10.1. With Lucene 10.1 we don't need this class
anymore.

Related to elastic#113554
@elasticsearchmachine elasticsearchmachine added the Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch label Jan 10, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-search-relevance (Team:Search Relevance)

@mayya-sharipova mayya-sharipova changed the title Remove temp XIntervals Remove temp XIntervals class Jan 10, 2025
Copy link
Contributor

@john-wagster john-wagster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mayya-sharipova
Copy link
Contributor Author

@elasticsearchmachine update branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>non-issue :Search Relevance/Search Catch all for Search Relevance Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants