-
Notifications
You must be signed in to change notification settings - Fork 35
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
Update support for csharp core implementation. #312
Conversation
config/samples/README.md
Outdated
Special considerations is required for running csharp examples, the support of | ||
csharp in [grc/grpc](https://github.com/grpc/grpc) is removed by | ||
<https://github.com/grpc/grpc/pull/29225>, csharp benchmark test is only | ||
supported by version v1.45.2 and earlier versions. |
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.
FYI C# is still available in gRPC core release v1.46.x (it's removed starting from 1.47.x i.e. the current master branch).
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 letting me know! Updated!
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.
LGTM once @paulosjca is happy.
* Update support for csharp core implementation. Co-authored-by: Paulo Castello da Costa <pcastello@google.com>
This commit adds the proper error message in the Dockerfiles
when user trying to build the csharp benchmark worker with
an incompatible grpc/grpc version.
This commit also add explanation for csharp example in
README.md in the config/sample folder.