-
Notifications
You must be signed in to change notification settings - Fork 780
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
Support Zstd compression for gRPC #5092
Conversation
Signed-off-by: Sesha Sendhil <seshasendhil@gmail.com>
Signed-off-by: Sesha Sendhil <seshasendhil@gmail.com>
Signed-off-by: Sesha Sendhil <seshasendhil@gmail.com>
This looks good! Can you update the changelog? |
Signed-off-by: Sesha Sendhil <seshasendhil@gmail.com>
Thanks @alanprot . Have updated the changelog |
LGTM |
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.
Thanks for the contribution!
CHANGELOG.md
Outdated
@@ -11,6 +11,7 @@ | |||
* [ENHANCEMENT] Query Frontend: Log Vertical sharding information when `query_stats_enabled` is enabled. #5037 | |||
* [ENHANCEMENT] Ingester: The metadata APIs should honour `querier.query-ingesters-within` when `querier.query-store-for-labels-enabled` is true. #5027 | |||
* [ENHANCEMENT] Query Frontend: Skip instant query roundtripper if sharding is not applicable. #5062 | |||
* [ENHANCEMENT] Added zstd as an option for grpc compression #5092 |
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.
Let's call it FEATURE
instead?
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.
Done @yeya24
Signed-off-by: Sesha Sendhil <seshasendhil@gmail.com>
* add zstd compression to grpcclient Signed-off-by: Sesha Sendhil <seshasendhil@gmail.com> * fix use of deprecated ioutil pkg Signed-off-by: Sesha Sendhil <seshasendhil@gmail.com> * update docs Signed-off-by: Sesha Sendhil <seshasendhil@gmail.com> * update CHANGELOG Signed-off-by: Sesha Sendhil <seshasendhil@gmail.com> * mark as feature Signed-off-by: Sesha Sendhil <seshasendhil@gmail.com> Signed-off-by: Sesha Sendhil <seshasendhil@gmail.com> Signed-off-by: Alex Le <leqiyue@amazon.com>
What this PR does:
Add zstd compression for grpc
Below output with zstd level BetterCompression as here
We see
Which issue(s) this PR fixes:
Fixes #5091
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]