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 #4519]Rust sdk support CloudEvents #4520

Merged
merged 3 commits into from
Nov 2, 2023

Conversation

mxsm
Copy link
Member

@mxsm mxsm commented Oct 30, 2023

Fixes #4519

Motivation

Explain the content here.
Explain why you want to make the changes and what problem you're trying to solve.

  • Rust sdk support CloudEvents

Modifications

Describe the modifications you've done.

  • Rust sdk grpc producer support CloudEvents
  • Optimize some code
  • Rename some rust source file name

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

How to use see producer_example.rs, Run result

2023-10-30T09:25:46.464595Z  INFO main ThreadId(01) producer_example: 71: Publish Message to EventMesh........
2023-10-30T09:25:46.570835Z  INFO main ThreadId(01) producer_example: 81: Publish Message to EventMesh return result: EventMeshResponse[code=0,message=success SendRe
sult[topic=TEST-TOPIC-GRPC-ASYNC, messageId=7F0000016EF018B4AAC29915DBC60019],response time=0,]
2023-10-30T09:25:46.571124Z  INFO main ThreadId(01) producer_example: 84: Publish batch message to EventMesh........
2023-10-30T09:25:46.587213Z  INFO main ThreadId(01) producer_example: 87: Publish batch message to EventMesh return result: EventMeshResponse[code=0,message=success 
batch publish success,response time=0,]

@mxsm mxsm requested review from xwm1992 and qqeasonchen October 30, 2023 09:26
@codecov
Copy link

codecov bot commented Oct 30, 2023

Codecov Report

Merging #4520 (815088e) into master (11def17) will not change coverage.
The diff coverage is n/a.

❗ Current head 815088e differs from pull request most recent head 248be16. Consider uploading reports for the commit 248be16 to get more accurate results

@@            Coverage Diff            @@
##             master    #4520   +/-   ##
=========================================
  Coverage     15.90%   15.90%           
  Complexity     1541     1541           
=========================================
  Files           728      728           
  Lines         28884    28884           
  Branches       2744     2744           
=========================================
  Hits           4593     4593           
  Misses        23811    23811           
  Partials        480      480           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@lrhkobe lrhkobe left a comment

Choose a reason for hiding this comment

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

LGTM

@lrhkobe lrhkobe merged commit 40a2aea into apache:master Nov 2, 2023
@mxsm mxsm deleted the feature-rust-4519 branch November 2, 2023 02:17
@pandaapo pandaapo added this to the 1.10 milestone Dec 5, 2023
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.

[Feature] Rust sdk support CloudEvents
4 participants