Skip to content

perf(store): optimize UpdateReadSet allocation pattern#2837

Merged
pdrobnjak merged 0 commit intoperf/hybrid-syncmap-rwmutexfrom
perf/optimize-updatereadset-alloc
Feb 11, 2026
Merged

perf(store): optimize UpdateReadSet allocation pattern#2837
pdrobnjak merged 0 commit intoperf/hybrid-syncmap-rwmutexfrom
perf/optimize-updatereadset-alloc

Conversation

@pdrobnjak
Copy link
Contributor

Summary

  • Optimize allocation pattern in UpdateReadSet to reduce GC pressure

Stack

2/13 — depends on perf/hybrid-syncmap-rwmutex

🤖 Generated with Claude Code

@github-actions
Copy link

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedFeb 11, 2026, 12:52 PM

@codecov
Copy link

codecov bot commented Feb 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.66%. Comparing base (2b3cec4) to head (b97b114).

Additional details and impacted files

Impacted file tree graph

@@                       Coverage Diff                       @@
##           perf/hybrid-syncmap-rwmutex    #2837      +/-   ##
===============================================================
- Coverage                        56.68%   56.66%   -0.03%     
===============================================================
  Files                             2031     2031              
  Lines                           165988   165990       +2     
===============================================================
- Hits                             94092    94052      -40     
- Misses                           63664    63697      +33     
- Partials                          8232     8241       +9     
Flag Coverage Δ
sei-chain 41.52% <66.66%> (-0.01%) ⬇️
sei-cosmos 48.17% <100.00%> (+<0.01%) ⬆️
sei-db 68.72% <ø> (ø)
sei-tendermint 58.05% <ø> (-0.08%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
sei-cosmos/store/multiversion/mvkv.go 90.90% <100.00%> (+0.11%) ⬆️

... and 29 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pdrobnjak pdrobnjak merged commit de1b10e into perf/hybrid-syncmap-rwmutex Feb 11, 2026
@pdrobnjak pdrobnjak force-pushed the perf/hybrid-syncmap-rwmutex branch from 2b3cec4 to f8b8fc9 Compare February 11, 2026 13:19
@pdrobnjak pdrobnjak deleted the perf/optimize-updatereadset-alloc branch February 11, 2026 13:19
@pdrobnjak pdrobnjak force-pushed the perf/optimize-updatereadset-alloc branch from b97b114 to de1b10e Compare February 11, 2026 13:19
@pdrobnjak pdrobnjak restored the perf/optimize-updatereadset-alloc branch February 11, 2026 13:22
@pdrobnjak
Copy link
Contributor Author

This PR was auto-merged by GitHub during a stack reorder (branch force-push made head appear merged into base). No code was actually merged to main. This PR has been superseded — see the restacked PR chain starting at #2849.

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.

1 participant

Comments