Skip to content

Postings writer codex + inverted index reader as trait#2819

Open
PSeitz-dd wants to merge 8 commits intomainfrom
postings-writer-enum-codec5
Open

Postings writer codex + inverted index reader as trait#2819
PSeitz-dd wants to merge 8 commits intomainfrom
postings-writer-enum-codec5

Conversation

@PSeitz-dd
Copy link
Contributor

@PSeitz-dd PSeitz-dd commented Jan 29, 2026

Checked

  • Search Performance (search benchmark game)
  • Merge Performance (benches/merge_segments.rs)

@PSeitz-dd PSeitz-dd changed the title Postings writer codex + inverted index as trait Postings writer codex + inverted index reader as trait Jan 29, 2026
@PSeitz-dd PSeitz-dd force-pushed the postings-writer-enum-codec5 branch 5 times, most recently from aa480a7 to e50407a Compare February 4, 2026 11:10
@fulmicoton fulmicoton self-requested a review February 11, 2026 10:28
@PSeitz-dd PSeitz-dd force-pushed the postings-writer-enum-codec5 branch from e50407a to 6b59e2b Compare February 11, 2026 14:07
Convert SegmentReader, InvertedIndexReader and postinglists to traits.
Add special functions to pushdown certain performance methods to keep
them strictly typed.

We rely on a ObjectSafeCodec contraption to avoid the proliferation of generics.
That object's point is to make sure we can build TermScorer with a concrete
codec specific type before reboxing it. (same thing for PhraseScorer).

fix performance regression: fix incorrect scorer cast for buffered union
bock wand
@PSeitz-dd PSeitz-dd force-pushed the postings-writer-enum-codec5 branch from 6b59e2b to 6038455 Compare February 11, 2026 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants