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

Add rate limiter for partial uploads #8203

Merged
merged 1 commit into from
Jun 28, 2024
Merged

Conversation

petuhovskiy
Copy link
Member

@petuhovskiy petuhovskiy commented Jun 28, 2024

Too many concurrect partial uploads can hurt disk performance, this commit adds a limiter.

Context: https://neondb.slack.com/archives/C04KGFVUWUQ/p1719489018814669?thread_ts=1719440183.134739&cid=C04KGFVUWUQ

@petuhovskiy petuhovskiy requested a review from a team as a code owner June 28, 2024 14:29
@petuhovskiy petuhovskiy requested review from jcsp and arssher June 28, 2024 14:29
Copy link

github-actions bot commented Jun 28, 2024

2940 tests run: 2823 passed, 0 failed, 117 skipped (full report)


Flaky tests (1)

Postgres 14

  • test_timeline_size_quota_on_startup: release

Code coverage* (full report)

  • functions: 32.7% (6909 of 21120 functions)
  • lines: 50.1% (54149 of 108105 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
81cfbe7 at 2024-06-28T15:22:13.310Z :recycle:

@petuhovskiy petuhovskiy merged commit e1a06b4 into main Jun 28, 2024
66 of 70 checks passed
@petuhovskiy petuhovskiy deleted the sk-limit-partial-uploads branch June 28, 2024 17:16
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.

None yet

4 participants