Skip to content

[FIXED] Filestore MaxMsgs/Bytes purge block accounting#7685

Merged
neilalexander merged 1 commit intomainfrom
maurice/fs-purge-mb
Jan 1, 2026
Merged

[FIXED] Filestore MaxMsgs/Bytes purge block accounting#7685
neilalexander merged 1 commit intomainfrom
maurice/fs-purge-mb

Conversation

@MauriceVanVeen
Copy link
Copy Markdown
Member

The filestore optimizes for large purges based on MaxMsgs/MaxBytes if full blocks can be removed in one go. However, these removed blocks were not accounted for under the JetStream usage stats.

Signed-off-by: Maurice van Veen github@mauricevanveen.com

Signed-off-by: Maurice van Veen <github@mauricevanveen.com>
@MauriceVanVeen MauriceVanVeen requested a review from a team as a code owner December 30, 2025 19:04
Copy link
Copy Markdown
Member

@neilalexander neilalexander left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@neilalexander neilalexander merged commit 6cfa943 into main Jan 1, 2026
130 of 136 checks passed
@neilalexander neilalexander deleted the maurice/fs-purge-mb branch January 1, 2026 12:11
neilalexander added a commit that referenced this pull request Jan 6, 2026
Includes the following:

- #7565
- #7589
- #7600
- #7602
- #7609
- #7610
- #7632
- #7649
- #7642
- #7658
- #7659
- #7661
- #7662
- #7663
- #7668
- #7683
- #7685
- #7686
- #7678
- #7691
- #7696
- #7698
- #7699
- #7700

Signed-off-by: Neil Twigg <neil@nats.io>
neilalexander added a commit that referenced this pull request Jan 6, 2026
Includes the following:

- #7658
- #7659
- #7661
- #7662
- #7663
- #7642
- #7668
- #7683
- #7685
- #7686
- #7678
- #7691
- #7696
- #7697
- #7698
- #7699
- #7700

Signed-off-by: Neil Twigg <neil@nats.io>
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.

2 participants