blevesearch / bleve

52%

master: 52%

LAST BUILD ON BRANCH bitpack-simd

branch: bitpack-simd

CHANGE BRANCH

x

Reset

• bitpack-simd
• BQMain
• HEAD
• MB-65018
• MB-71114-knn-doc-score-explanation-fix
• MB-71967-search-field-override-fix
• MB-71968
• MB-72535
• MB-73497
• ajroetker/add-dynamic-index-templates
• ajroetker/add-more-complex-aggregation-support
• ajroetker/add-result-deduplication-aka-collapsing
• alice
• backport_MB-37392
• backport_json_tags
• backport_merger_fix
• backport_polygon1
• backport_polygon_changes
• batchFlush
• blas
• bolt
• boltLeak
• bugFix
• cacheRootCount
• close_err
• commitProtocol
• compaction_bytes_stats
• conditional_snappy
• config_fix
• cosineFix
• coveralls
• cq
• ctl-timeout-test
• custom-query-error-propagation
• custom_expl
• cut_snappy_calls
• defensiveChecks
• dependabot/go_modules/golang.org/x/net-0.55.0
• deps
• dev
• doc-260
• docnum_missing_fix
• docs
• drop
• encryptTrainedIndex
• error_fuzzy
• faissUP
• fallbackPath
• fastbivf
• fastmerge
• fileCallbacks
• fileTransfer
• filterFix
• firestorm
• fix-2362-bare-slash-query
• fixes
• fmdoc
• funcFix
• geo
• geoPerf
• geopoint_docvalues
• gha-hygiene
• goModUp
• gofmt
• govariant_removal
• gpu
• gpu-logging
• gpu-stats
• gpuDoc
• indexAPI
• indexExhaust
• knnDup
• loadchunk_minor
• mad-hatter-release
• master
• master-geopoint
• max_segment_size
• mb70770
• mergePlan
• metareader
• minor_docvalue_space_savings
• minor_fixes
• mod
• multisearch-fix
• nested
• nestedDocSize
• new_lev
• newlev_adoption
• nullIssue
• numsnapshots_config
• open_fix
• parMerge
• perf
• perf-fuzzy-regexp-opts
• perf-gar-v17-collector
• perf-standalone-opts
• perfGar1
• persister_fix
• pre-bbolt-bugfix
• repos
• rollback
• scorch
• scorchStatsFix
• shapeIndex
• simplifyFilter
• single-disjunction-min
• snap
• stats
• temp
• tfr_advance
• trainingParams
• travis_failure_fix
• unit-tests
• update
• upgrade
• v0.1.0
• v0.2.0
• v0.3.0
• v0.4.0
• v0.5.0
• v0.6.0
• v0.7.0
• v0.8.0
• v0.8.1
• v0.8.2
• v0.9.0
• v1.0.5
• vectorsmd
• vellum_for_fuzzy_regex
• vellum_sha
• vendot
• verUp
• vet_fixes
• vulcan
• zapConfig
• zapUP
• zapUp
• zap_version

Build # 35038620957

Build Type

Pull #2398

github

Committed by capemox

Commit Message

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>

Pull Request Pull Request #2398: Replace prealloc-passing with Release()-based pooling for term postings

Coverage Stats

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