Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[v23.1.x] storage: shrink index vectors on flush #10043

Merged
merged 1 commit into from
Apr 14, 2023
Merged

[v23.1.x] storage: shrink index vectors on flush #10043

merged 1 commit into from
Apr 14, 2023

Conversation

jcsp
Copy link
Contributor

@jcsp jcsp commented Apr 13, 2023 •
edited
Loading

Backport: #9904

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v23.1.x
  • v22.3.x
  • v22.2.x

Release Notes

Improvements

  • Reduced memory consumption on configurations with very large numbers of small segments.

@jcsp
storage: shrink index vectors on flush
a6c3b53 
This saves up to 24kib of memory per index, where
a segment is closed with very few entries in its index.

Related: redpanda-data#9375
(cherry picked from commit f5e6b57)
@jcsp jcsp added area/storage kind/backport PRs targeting a stable branch labels Apr 13, 2023
@jcsp jcsp changed the title storage: shrink index vectors on flush [v23.1.x] storage: shrink index vectors on flush Apr 14, 2023
@jcsp jcsp merged commit 8ce86e4 into redpanda-data:v23.1.x Apr 14, 2023
@jcsp jcsp deleted the backport-v23.1.x-9904 branch April 14, 2023 08:53
@BenPope BenPope added this to the v23.1.7 milestone Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
area/redpanda area/storage kind/backport PRs targeting a stable branch
Projects
None yet
Milestone
v23.1.7
Development

Successfully merging this pull request may close these issues.
2 participants
@jcsp @BenPope