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.

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

Comprehensive Limiters for IO and CPU #1720

Closed
flowbehappy opened this issue Apr 2, 2021 · 3 comments · Fixed by #3135
Closed

Comprehensive Limiters for IO and CPU #1720

flowbehappy opened this issue Apr 2, 2021 · 3 comments · Fixed by #3135
Assignees
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@flowbehappy
Copy link
Contributor

Currently, we only have IO limiters for background tasks. Which is not enough. For example, a heavy read task can consume lots of IO, and slow down the write processes, eventually cause the write stall or high Wait Index Duration.

Ideally, we should have limiters, both IO and CPU, for

  • Read threads
  • Frontend write threads
  • Background write threads

And the threshold is better to be self-adjusted. Since CPU is known to be very complicated to limit, we can start with IO limiters.

@flowbehappy flowbehappy added the type/enhancement The issue or PR belongs to an enhancement. label Apr 2, 2021
@JinheLin
Copy link
Contributor

JinheLin commented Apr 30, 2021

@JinheLin
Copy link
Contributor

Subtask(Plan)

  • Implement configurable I/O rate limit for background and foreground write request. (doing: deving and self-testing)
  • Use epbf to monitoring the I/O statistics, such as read/write bandwith and latency.
  • Maybe use Block IO Controller to limit the total read I/O bandwith.
  • Try to implement the auto-tune IO rate limiter.

@JaySon-Huang
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants