Skip to content
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

Merged
merged 3 commits into from
May 31, 2022

Conversation

wanlin31
Copy link
Collaborator

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.

@wanlin31 wanlin31 marked this pull request as ready for review May 25, 2022 01:47
@wanlin31 wanlin31 requested a review from paulosjca May 25, 2022 01:47
@wanlin31 wanlin31 self-assigned this May 25, 2022
@wanlin31 wanlin31 added the release notes: yes Indicates that PR needs to be in release notes label May 25, 2022
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.
Copy link
Collaborator

@jtattermusch jtattermusch May 25, 2022

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).

Copy link
Collaborator Author

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!

Copy link
Collaborator

@jtattermusch jtattermusch left a 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.

@wanlin31 wanlin31 merged commit b3dc202 into grpc:master May 31, 2022
wanlin31 added a commit to wanlin31/test-infra that referenced this pull request Jun 15, 2022
* Update support for csharp core implementation.

Co-authored-by: Paulo Castello da Costa <pcastello@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release notes: yes Indicates that PR needs to be in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants