Skip to content
This repository has been archived by the owner on Jun 23, 2022. It is now read-only.

feat: support nfs server rate limit #901

Merged
merged 11 commits into from
Sep 6, 2021
Merged

feat: support nfs server rate limit #901

merged 11 commits into from
Sep 6, 2021

Conversation

foreverneverer
Copy link
Contributor

@foreverneverer foreverneverer commented Sep 3, 2021 •
edited
Loading

In some test, we found that the influence usually caused by nfs_server side under learning copy checkpoint. this pr support the nfs server send rate limit.

Note

  • limit server send rate may casuse the rpc timeout,so this pr update nfs rpc timeout from 10s to 100s
  • update max_send_rate via admin-cli server-config may be block for a long time, this pr update the task priority
  • for compatibility with remote-command, this pr support it too

@foreverneverer foreverneverer marked this pull request as ready for review September 3, 2021 10:07
@Smityz Smityz added type/config-change PR that made modification on configs, which should be noted in release note. type/performance Issues or PRs related to performacne labels Sep 3, 2021
@hycdong hycdong merged commit 2cec889 into XiaoMi:master Sep 6, 2021
@hycdong hycdong mentioned this pull request Sep 23, 2021
Closed
@foreverneverer foreverneverer mentioned this pull request Dec 4, 2021
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@levy5307 levy5307 levy5307 approved these changes

@hycdong hycdong hycdong approved these changes

Assignees
No one assigned
Labels
type/config-change PR that made modification on configs, which should be noted in release note. type/performance Issues or PRs related to performacne
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@foreverneverer @hycdong @levy5307 @Smityz