-
Notifications
You must be signed in to change notification settings - Fork 203
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
Try replace grpcio
with tonic
#240
Closed
Labels
feature
New feature or request
Comments
This was referenced Sep 23, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe This Problem
grpcio
is currently used by CeresDB as gRPC implementation. It's a rust wrapper over the cpp implementation. We have used it for a long time but there are always some issues we are facing:tonic
is a native gRPC implementation with these pros.:grpcio
)And of cause, its cons.:
Performance (refer to https://github.com/LesnyRumcajs/grpc_bench/wiki/2022-04-23-bench-results)
through this table,
grpcio
is faster thantonic
. QpS on gRPC layer loss around 7%.Proposal
With those improvements, I think
tonic
is worth a tryAdditional Context
No response
The text was updated successfully, but these errors were encountered: