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

[ISSUE #4845] Update grpc version from 1.43.2 to 1.63.0 #4876

Open
wants to merge 19 commits into
base: master
Choose a base branch
from

Conversation

pmupkin
Copy link
Contributor

@pmupkin pmupkin commented May 15, 2024

Fixes #4845

Motivation

update grpc version and protobuf version

Modifications

Describe the modifications you've done.

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
  • If a feature is not applicable for documentation, explain why?
  • If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

Copy link
Member

@mxsm mxsm left a comment

Choose a reason for hiding this comment

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

@pmupkin Branch has conflicts that must be resolved. Please fix it

@pmupkin
Copy link
Contributor Author

pmupkin commented May 15, 2024

@pmupkin Branch has conflicts that must be resolved. Please fix it

OK

Comment on lines 23 to 25
eventMesh.sysid=0000
eventMesh.server.tcp.port=10000
eventMesh.server.tcp.port=10002
eventMesh.server.http.port=10105
Copy link
Member

Choose a reason for hiding this comment

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

Please restore these changes.

Comment on lines 72 to 77
# storage plugin
eventMesh.storage.plugin.type=standalone
eventMesh.storage.plugin.type=rocketmq

##connector plugin
#eventMesh.connector.plugin.type=rocketmq

Copy link
Member

Choose a reason for hiding this comment

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

Please restore these changes.

install.sh Outdated
Comment on lines 28 to 32
gradle clean -Pdev=true -Pjdk=1.8 dist tar zip

gradlew installPlugin
# package jar
gradle clean -Pdev=true -Pjdk=1.8 jar No newline at end of file
#gradle clean -Pdev=true -Pjdk=1.8 jar
Copy link
Member

Choose a reason for hiding this comment

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

This script seems to be getting old, you can just delete this script. If you want to build the project, please refer to this documentation.

Comment on lines 26 to 29

def grpcVersion = '1.43.2' // CURRENT_GRPC_VERSION
def protobufVersion = '3.21.5'
def grpcVersion = '1.63.0' // CURRENT_GRPC_VERSION
def protobufVersion = '3.25.1'
def protocVersion = protobufVersion
Copy link
Member

Choose a reason for hiding this comment

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

I think we just need to define grpcVersion and protobufVersion once in the build.gradle file in the project root directory.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I also think it's more appropriate to define it under the root directory. I will resubmit to fix the above issues.

Copy link

codecov bot commented May 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 16.47%. Comparing base (70f9892) to head (f1e5531).
Report is 26 commits behind head on master.

Current head f1e5531 differs from pull request most recent head 9a51e8b

Please upload reports for the commit 9a51e8b to get more accurate results.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #4876      +/-   ##
============================================
+ Coverage     15.91%   16.47%   +0.56%     
- Complexity     1734     1819      +85     
============================================
  Files           897      910      +13     
  Lines         31982    32363     +381     
  Branches       2737     2787      +50     
============================================
+ Hits           5089     5331     +242     
- Misses        26413    26520     +107     
- Partials        480      512      +32     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Pil0tXia
Copy link
Member

Please resolve conflicts.

@Pil0tXia
Copy link
Member

May I ask that is this PR ready for review?

Copy link
Member

@Pil0tXia Pil0tXia left a comment

Choose a reason for hiding this comment

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

Why can't grpcVersion be managed directly in dependencyManagement, but needs to use the ext field?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Enhancement] Update grpc version from 1.43.2 to 1.63.0
3 participants