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

Sync commits #64

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

Conversation

coco-speed
Copy link
Collaborator

This pull request was created automatically by CodSpeed to track performance changes of the pull request cloudflare/pingora#348.

The original branch is upstream/bleeper-andrewhauck-f123f5e43e

eaufavor and others added 13 commits July 25, 2024 11:31
Before this change, the retry only happens on reused connections. The
shutting down could also happen on new connections as well. So this
change will retry in both cases.
---
Specified quick_cache version because older version was being loaded on build

Includes-commit: 3f5419e
Includes-commit: de83f0a
Replicated-from: cloudflare#248
There could be a race condition where a Stale request waiting on cache
lock retries the cache lock and lookup, but enters cache_miss after
finding the asset is no longer present.
---
Update docs/user_guide/ratelimiter.md
---
docs: add upstream_request_filter to fix forbidden error of RateLimiter
---
Update ratelimiter.md

update rate_limiter_map initialize code

Co-authored-by: Dany Gagnon <[email protected]>
Co-authored-by: Xia Lei <[email protected]>
Includes-commit: 461d384
Includes-commit: 8023171
Includes-commit: 99ed87f
Includes-commit: bffc9f4
Replicated-from: cloudflare#120
---
use weight as param
---
fix format and doc comment
---
add ext as Extensions::new
---
Merge branch 'main' into ~~~

Includes-commit: 0d4a658
Includes-commit: 1ee9b32
Includes-commit: 5b05294
Includes-commit: cf8faea
Includes-commit: f0cefff
Replicated-from: cloudflare#333
This cache phase indicates when we are serving a stale cache hit, but
there is another request currently updating the asset.
---
test: add test for upstream health observe
---
renamed the function and added doc to make it intelligible
---
fix clippy error
---
Merge branch 'main' into main
---
test: fix test for backend do update

Co-authored-by: Tree Xie <[email protected]>
Includes-commit: 1421c26
Includes-commit: 695d549
Includes-commit: 6a09b52
Includes-commit: 72d6ee0
Includes-commit: e6c2af0
Includes-commit: fb62869
Replicated-from: cloudflare#325
Copy link

codspeed-hq bot commented Aug 9, 2024

CodSpeed Performance Report

Merging #64 will not alter performance

Comparing upstream/bleeper-andrewhauck-f123f5e43e (4ee1e90) with main (ca93004)

Summary

✅ 2 untouched benchmarks

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.

9 participants