Skip to content

Disk buffer abandoned disk-buffer-metrics#902

Merged
MrAnno merged 1 commit intoaxoflow:mainfrom
bshifter:disk-buffer-abandoned-disk-metrics
Jan 19, 2026
Merged

Disk buffer abandoned disk-buffer-metrics#902
MrAnno merged 1 commit intoaxoflow:mainfrom
bshifter:disk-buffer-abandoned-disk-metrics

Conversation

@bshifter
Copy link
Member

@bshifter bshifter commented Jan 13, 2026

Previously, abandoned disk buffer metrics were calculated by loading the entire buffer file, which could significantly impact reload times. This change avoids loading the full disk-buffer contents by using the information already available in the file header, which is sufficient to compute the required metrics.

Depends on #912

@bshifter bshifter force-pushed the disk-buffer-abandoned-disk-metrics branch from dbc2863 to 66a9232 Compare January 13, 2026 09:36
@bshifter bshifter changed the title Disk buffer abandoned disk metrics Disk buffer abandoned disk-buffer-metrics Jan 13, 2026
@MrAnno MrAnno self-requested a review January 14, 2026 11:09
@bshifter bshifter force-pushed the disk-buffer-abandoned-disk-metrics branch 2 times, most recently from 90a8fb8 to 644cf91 Compare January 15, 2026 13:25
@MrAnno
Copy link
Contributor

MrAnno commented Jan 15, 2026

Please don't forget to write a bugfix news entry both for #901 and #902.

Copy link
Member

@bazsi bazsi left a comment

Choose a reason for hiding this comment

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

This needs a rebase now that #901 is in, looks good to me.

@bshifter bshifter force-pushed the disk-buffer-abandoned-disk-metrics branch 2 times, most recently from 6f2ae1c to 7a7dd77 Compare January 16, 2026 13:54
MrAnno
MrAnno previously approved these changes Jan 16, 2026
@bshifter bshifter dismissed MrAnno’s stale review January 16, 2026 15:52

The merge-base changed after approval.

@bshifter bshifter force-pushed the disk-buffer-abandoned-disk-metrics branch from 7a7dd77 to ae40ddf Compare January 16, 2026 16:14
@MrAnno
Copy link
Contributor

MrAnno commented Jan 19, 2026

@mitzkia may have found some issues in #903.

@mitzkia
Copy link
Contributor

mitzkia commented Jan 19, 2026

@mitzkia may have found some issues in #903.

Ok, I have checked new disk-buffer testscases for this branch , they are passed.

@MrAnno MrAnno merged commit 367c017 into axoflow:main Jan 19, 2026
21 of 22 checks passed
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.

4 participants