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

snapshotcloneserver: add s3 throttle #303

Merged
merged 1 commit into from
Mar 31, 2021
Merged

Conversation

xu-chaojie
Copy link
Member

@xu-chaojie xu-chaojie commented Mar 26, 2021

What problem does this PR solve?

Issue Number: close #xxx

Problem Summary:

What is changed and how it works?

What's Changed:

How it Works:

Side effects(Breaking backward compatibility? Performance regression?):

Check List

  • Relevant documentation/comments is changed or added
  • I acknowledge that all my contributions will be made under the project's license

@xu-chaojie xu-chaojie force-pushed the master branch 2 times, most recently from 760310c to 61b7a39 Compare March 26, 2021 03:09
* 从对象存储读取数据,
* 注:本函数仅用于控制面获取少量数据,只对qps限流,没有对带宽的限流,
* 数据面使用下面两个接口
*
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

english

@@ -15,3 +15,10 @@ s3.request_timeout=10000
# Off = 0,Fatal = 1,Error = 2,Warn = 3,Info = 4,Debug = 5,Trace = 6
s3.loglevel=4
s3.async_thread_num=64
# throttle
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

snapshotcloneserver: add s3 throttle

@xu-chaojie xu-chaojie changed the title add s3 throttle snapshotcloneserver: add s3 throttle Mar 31, 2021
@xu-chaojie xu-chaojie merged commit c823c57 into opencurve:master Mar 31, 2021
ilixiaocui pushed a commit to ilixiaocui/curve that referenced this pull request Feb 6, 2023
Selection for OpenTelemetry FluentBit project
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.

3 participants