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

feat: 升级grpc扩展中grpc版本到1.62.2 #1084

Merged
merged 1 commit into from
Mar 7, 2024
Merged

feat: 升级grpc扩展中grpc版本到1.62.2 #1084

merged 1 commit into from
Mar 7, 2024

Conversation

zxuanhong
Copy link
Contributor

@zxuanhong zxuanhong commented Mar 7, 2024

  1. 升级grpc版本到1.62.2
  2. 添加grpc util包(原grpc core util抽离为单独的依赖了)

Motivation:

  1. 目前jraft对jdk高版本支持还有问题,主要出现在rpc模块。jdk21下操作CliService会导致Connection is null when do check!。
  2. 经测试发现使用grpc能很好解决这个问题。但仓库使用grpc版本太低,进行升级到最新版

Modification:

高版本grpc core util相关代码已经完全抽离出去了,对grpc依赖进行升级,同时引入缺少的grpc util包

Result:

Fixes #.

If there is no issue then describe the changes introduced by this PR.

1. 升级grpc版本到1.62.2
2. 添加grpc util包(原grpc core util抽离为单独的依赖了)
Copy link

sofastack-cla bot commented Mar 7, 2024

Hi @zxuanhong, welcome to SOFAStack community, Please sign Contributor License Agreement!

After you signed CLA, we will automatically sync the status of this pull request in 3 minutes.

@zxuanhong zxuanhong changed the title feat: 升级grpc实现中grpc版本到1.62.2 feat: 升级grpc扩展中grpc版本到1.62.2 Mar 7, 2024
@fengjiachun
Copy link
Contributor

@zxuanhong Hi, CLA 需要签一下

@zxuanhong
Copy link
Contributor Author

@fengjiachun 抱歉点击后没注意失败了。已经再次处理

@zhenjunMa zhenjunMa closed this Mar 7, 2024
@zhenjunMa zhenjunMa reopened this Mar 7, 2024
@sofastack-cla sofastack-cla bot added the cla:yes label Mar 7, 2024
Copy link
Contributor

@fengjiachun fengjiachun left a comment

Choose a reason for hiding this comment

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

LGTM

@fengjiachun fengjiachun merged commit 890033a into sofastack:master Mar 7, 2024
17 checks passed
@fengjiachun
Copy link
Contributor

@zxuanhong Thanks

@shihuili1218 shihuili1218 mentioned this pull request Aug 7, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants