blevesearch / bleve / 35038620957

52%

master: 52%

Pull #2398

github

Replace prealloc-passing with Release()-based pooling for term postings

TermFieldReader() and prepareField() used to hand rv.postings[i]/
rv.iterators[i] back in as a same-shaped "prealloc" hint on every
lookup, but a recycled reader's shape is frequently wrong (a mix of
1-hit and general-encoded terms on the same field), so the hint was
often discarded and allocated over anyway.

Both call sites now release a value they're done with via
releasePostings() -- a type assertion against the new, optional
segment.Releasable capability (scorch_segment_api bitpack-simd
branch), the same opt-in idiom as segment.OptimizablePostingsIterator
-- and fetch a fresh one with prealloc left nil, letting the segment
implementation's own pool decide what to hand back. Values that don't
implement Releasable (e.g. the unadorned/synthetic iterators built by
query optimization) make this a no-op, same as before.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

12 of 16 new or added lines in 2 files covered. (75.0%)

20161 of 38984 relevant lines covered (51.72%)

0.58 hits per line