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

feat: 添加限流模块 #1539 #1829

Merged
merged 68 commits into from
Nov 5, 2024
Merged

Conversation

zacYL
Copy link
Collaborator

@zacYL zacYL commented Mar 4, 2024

No description provided.

@zacYL zacYL requested a review from owenlxu as a code owner March 15, 2024 11:23
@zacYL zacYL self-assigned this Mar 18, 2024
@owenlxu owenlxu removed the request for review from felixncheng June 18, 2024 07:20
@owenlxu owenlxu added the wip work in progess label Jun 18, 2024
yaoxuwan
yaoxuwan previously approved these changes Nov 1, 2024
@cnlkl
Copy link
Collaborator

cnlkl commented Nov 4, 2024

目前的限流方案会导致用户下载一半由于限流下载失败,如果多个用户一起下载,用户每次失败后都重试可能导致所有用户均无法下载成功

例如:

用户A:下载中 -> 下载中 -> 下载中 -> 被限流 -> 重试
用户B:下载中 -> 被限流 -> 重试 -> 下载中 -> 下载中

cnlkl
cnlkl previously approved these changes Nov 4, 2024
@zacYL zacYL requested review from cnlkl and yaoxuwan November 4, 2024 09:38
@zacYL zacYL dismissed stale reviews from cnlkl and yaoxuwan via 1e8e153 November 4, 2024 12:08
cnlkl
cnlkl previously approved these changes Nov 4, 2024
yaoxuwan
yaoxuwan previously approved these changes Nov 4, 2024
@zacYL zacYL dismissed stale reviews from yaoxuwan and cnlkl via 1d0fbe2 November 5, 2024 07:10
@owenlxu owenlxu merged commit 4401758 into TencentBlueKing:master Nov 5, 2024
5 of 6 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.

5 participants