-
Notifications
You must be signed in to change notification settings - Fork 481
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
Gateway: Adds header optimization #2098
Conversation
What's the RPS impact? Fully materialization and optimization are two improvements. |
...e.Cosmos/tests/Microsoft.Azure.Cosmos.EmulatorTests/FeedToken/ChangeFeedIteratorCoreTests.cs
Outdated
Show resolved
Hide resolved
…Azure/azure-cosmos-dotnet-v3 into users/jawilley/perf/httpHeaders
Still running RPS for direct mode to see if there is an impact. Gateway mode RPS is described in the PR description with roughly 5.6% gain. |
public override int Count() | ||
{ | ||
int count = 0; | ||
if (this.dictionaryNameValueCollection.IsValueCreated) |
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.
Nice:smile:
…Azure/azure-cosmos-dotnet-v3 into users/jawilley/perf/httpHeaders
Pull Request Template
Description
This adds a optimized header for the gateway store model. This model wraps the existing header objects. This avoids iterating and doing string join on all the operations.
Top10AVGRps for gateway mode
Master: 11868.333333
Optimized: 12555.411765
Profiler shows it consumes %16 of CPU from code in main branch. After this PR the profiler no longer shows the conversion.
![MicrosoftTeams-image (1)](https://user-images.githubusercontent.com/8868107/103294886-90a48800-49a7-11eb-861d-dd973e26b3e1.png)
Type of change
Please delete options that are not relevant.
Closing issues
To automatically close an issue: closes #IssueNumber