Skip to content

[revert] "[improvement]upgrade-grpc-version"#8250

Merged
morningman merged 1 commit intomasterfrom
revert-8218-upgrade-grpc-version
Mar 2, 2022
Merged

[revert] "[improvement]upgrade-grpc-version"#8250
morningman merged 1 commit intomasterfrom
revert-8218-upgrade-grpc-version

Conversation

@morningman
Copy link
Contributor

Reverts #8218

Because when using grpc 1.44.1, the corresponding protoc-gen-grpc-java plugin
requried GLIBC_2.14, which is not found in CentOS 6.

So I suggest to revert this commit this time. And considering upgrading this component
after most systems have reached glibc version 2.14.

And for Mac M1, you may have to change this version manually for now

@morningman morningman added dev/backlog waiting to be merged in future dev branch kind/revert Revert a previous commit labels Feb 26, 2022
@morningman
Copy link
Contributor Author

CC @wunan1210

Copy link
Contributor

@hf200012 hf200012 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions
Copy link
Contributor

github-actions bot commented Mar 1, 2022

PR approved by at least one committer and no changes requested.

@github-actions github-actions bot added approved Indicates a PR has been approved by one committer. reviewed labels Mar 1, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Mar 1, 2022

PR approved by anyone and no changes requested.

@morningman morningman merged commit 315bfe2 into master Mar 2, 2022
morningman added a commit that referenced this pull request Mar 14, 2022
This reverts commit df7e848.

Reverts #8218

Because when using grpc 1.44.1, the corresponding `protoc-gen-grpc-java` plugin
requried GLIBC_2.14, which is not found in CentOS 6.

So I suggest to revert this commit this time. And considering upgrading this component
after most systems have reached glibc version 2.14.

And for Mac M1, you may have to change this version manually for now
@morningman morningman added dev/merged-1.0.1-deprecated PR has been merged into dev-1.0.1 and removed dev/backlog waiting to be merged in future dev branch labels Mar 14, 2022
@yangzhg yangzhg deleted the revert-8218-upgrade-grpc-version branch April 28, 2022 08:17
adonis0147 added a commit that referenced this pull request May 11, 2023
…9522)

1. add <optional> head to solve the compilation issue
use 3.12.9 as the protoc.artifact's version, because there is no 3.12.21
See: https://repo.maven.apache.org/maven2/com/google/protobuf/protoc/
2. Remove --show-progress arguments of wget because it is not supported in low version wget
3.  Downgrade grpc.version to 1.30.0 (See #8250).

Co-authored-by: Mingyu Chen <morningman@163.com>
mongo360 pushed a commit to mongo360/doris that referenced this pull request Jul 12, 2023
… (apache#19522)

1. add <optional> head to solve the compilation issue
use 3.12.9 as the protoc.artifact's version, because there is no 3.12.21
See: https://repo.maven.apache.org/maven2/com/google/protobuf/protoc/
2. Remove --show-progress arguments of wget because it is not supported in low version wget
3.  Downgrade grpc.version to 1.30.0 (See apache#8250).

Co-authored-by: Mingyu Chen <morningman@163.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. dev/merged-1.0.1-deprecated PR has been merged into dev-1.0.1 kind/revert Revert a previous commit reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants