Skip to content

Implement docIDRunEnd() on ES819TSDBDocValuesProducer #132939

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

Merged
merged 4 commits into from
Aug 15, 2025

Conversation

romseygeek
Copy link
Contributor

This method allows consumers to quickly check if a DocIdSetIterator matches a
large run of documents; it was missing from our custom Codec DocValues implementations.

This method allows consumers to quickly check if a DocIdSetIterator matches
a large run of documents; it was missing from our custom Codec DocValues
implementations.
Copy link

cla-checker-service bot commented Aug 14, 2025

💚 CLA has been signed

@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label external-contributor Pull request authored by a developer outside the Elasticsearch team labels Aug 14, 2025
@romseygeek romseygeek self-assigned this Aug 14, 2025
@romseygeek romseygeek added :StorageEngine/Codec and removed needs:triage Requires assignment of a team area label external-contributor Pull request authored by a developer outside the Elasticsearch team labels Aug 14, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-storage-engine (Team:StorageEngine)

@romseygeek
Copy link
Contributor Author

@elasticmachine test this please

@gtback
Copy link
Member

gtback commented Aug 14, 2025

cla/check

Copy link
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

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

LGTM

Somehow changelog entry wasn't added automatically. Not sure why.

Maybe label as non-issue? (then change log entry is not required)
Because this change comes with Lucene 10.3, currently stock doc values don't have still either. We caught up with this change in Lucene on time.

@romseygeek
Copy link
Contributor Author

@elasticmachine test this please

@romseygeek romseygeek assigned romseygeek and unassigned romseygeek Aug 15, 2025
@romseygeek
Copy link
Contributor Author

@elasticmachine update branch

@romseygeek
Copy link
Contributor Author

@elasticmachine update branch

@romseygeek romseygeek merged commit 5076681 into elastic:lucene_snapshot Aug 15, 2025
35 checks passed
@romseygeek romseygeek deleted the codec/docidendrun branch August 15, 2025 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants