-
Notifications
You must be signed in to change notification settings - Fork 15.6k
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
Expand benchmarks for Java #842
Comments
FYI, I'd be happy to do the work here. |
The benchmarks for proto Java are not opensourced. I suggest we track this bug internally, add new benchmarks to our internal code base and then get part of these benchmarks opensourced if needed. |
Also, for significant protobuf changes like this (the planned performance optimizations) we should do it internally first to make sure it does not impact google3 users negatively. |
@xfxyjwf sgtm ... I'll close this for now and continue internally. Thanks. |
From grpc/grpc-java#1054:
The gRPC team has a few ideas for optimizing protobuf marshalling but we need to expand the benchmark suite before that work can begin.
cc/ @louiscryan @ejona86 @xfxyjwf
The text was updated successfully, but these errors were encountered: