-
Notifications
You must be signed in to change notification settings - Fork 5.1k
Updated Default Concurrency count #54428
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
Updated Default Concurrency count #54428
Conversation
API Change CheckAPIView identified API level changes in this PR and created the following API reviews |
1 similar comment
API Change CheckAPIView identified API level changes in this PR and created the following API reviews |
|
|
||
| private readonly ArrayPool<byte> _arrayPool; | ||
|
|
||
| private readonly int DefaultConcurrentTransfersCount = Math.Min(Math.Max(Environment.ProcessorCount * 2, 8), 32); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Intentionally had to do this since Math.Clamp() is not supported for netstandard2.0 and earlier
jaschrep-msft
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nitpick suggestions. looks fine otherwise.
a389c3e
into
Azure:feature/storage/stg101base
Updated the default concurrency count. Also included compat switch logic to toggle back to previous default.