Skip to content

feat: upgrade foyer to v0.17, fix disk statistics export#64

Merged
tisonkun merged 3 commits intoscopedb:mainfrom
MrCroxx:xx/foyer-v0.17.0
May 8, 2025
Merged

feat: upgrade foyer to v0.17, fix disk statistics export#64
tisonkun merged 3 commits intoscopedb:mainfrom
MrCroxx:xx/foyer-v0.17.0

Conversation

@MrCroxx
Copy link
Contributor

@MrCroxx MrCroxx commented May 8, 2025

Changes:

  • Upgrade foyer to v0.17.
  • Fix disk statistics export.

Signed-off-by: MrCroxx <mrcroxx@outlook.com>
Copy link
Contributor

@tisonkun tisonkun left a comment

Choose a reason for hiding this comment

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

Seems previous io.count with read_label and flush_label are removed?

What's the rationale of this logical difference?

@MrCroxx
Copy link
Contributor Author

MrCroxx commented May 8, 2025

Seems previous io.count with read_label and flush_label are removed?

What's the rationale of this logical difference?

Only flush label is removed since flush mode (with fsync) is never used.

Signed-off-by: tison <wander4096@gmail.com>
Comment on lines 92 to 101
Copy link
Contributor

Choose a reason for hiding this comment

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

@MrCroxx BTW, what does "ios" stand for? Is it guaranteed to increase monotonically?

Copy link
Contributor

Choose a reason for hiding this comment

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

Otherwise, doing difference can cause usize/u64 underflow.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. It counts the IO request count. (To adopt to AWS EBS IO request count rule, it can be configured to count IO request by aligning to 128KiB.

Signed-off-by: tison <wander4096@gmail.com>
@tisonkun tisonkun enabled auto-merge (squash) May 8, 2025 08:25
@tisonkun tisonkun merged commit b2b0758 into scopedb:main May 8, 2025
4 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.

3 participants

Comments