Skip to content

Promote PerThreadPKLookup to core as PrimaryKeyLookup#16128

Open
shubhamsrkdev wants to merge 13 commits into
apache:mainfrom
shubhamsrkdev:PKLookup
Open

Promote PerThreadPKLookup to core as PrimaryKeyLookup#16128
shubhamsrkdev wants to merge 13 commits into
apache:mainfrom
shubhamsrkdev:PKLookup

Conversation

@shubhamsrkdev

@shubhamsrkdev shubhamsrkdev commented May 26, 2026

Copy link
Copy Markdown
Contributor

Description

  • Promoted PerThreadPKLookup to core as PrimaryKeyLookup (and deprecated PerThreadPKLookup)
  • Added Thread-stickiness guard
  • Added test cases for PrimaryKeyLookup

Fixes #15520

Comment thread lucene/core/src/java/org/apache/lucene/index/PrimaryKeyLookup.java
Comment thread lucene/core/src/java/org/apache/lucene/index/PrimaryKeyLookup.java Outdated
Comment thread lucene/core/src/java/org/apache/lucene/index/PrimaryKeyLookup.java
@shubhamsrkdev

Copy link
Copy Markdown
Contributor Author

Thanks for the review @epotyom ! If this looks ready can anyone help merge (or review) it? TIA!

@shubhamsrkdev

Copy link
Copy Markdown
Contributor Author

Gentle reminder - if anyone can help merge this it would be great, TIA!

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.

Should Lucene offer a helper class for primary key lookups?

2 participants