Skip to content

[ISSUE #4519]Rust sdk support CloudEvents #4520

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.

Sign up for GitHub

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

Jump to bottom
Merged
lrhkobe merged 3 commits into from
Nov 2, 2023
Merged

[ISSUE #4519]Rust sdk support CloudEvents #4520

lrhkobe merged 3 commits into from
Nov 2, 2023

Conversation

@mxsm
Copy link
Member

@mxsm mxsm commented Oct 30, 2023 •
edited
Loading

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 qqeasonchen and xwm1992 October 30, 2023 09:26
@codecov
Copy link

codecov bot commented Oct 30, 2023 •
edited
Loading

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

lrhkobe
lrhkobe approved these changes Nov 2, 2023
View reviewed changes
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
xuhongjia pushed a commit to Deckers-Ohana/eventmesh that referenced this pull request Mar 13, 2025
@mxsm
[ISSUE apache#4519]Rust sdk support CloudEvents (apache#4520)
8773102 
* [ISSUE apache#4519]Rust sdk support CloudEvents

* optimize code

* fix complie error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xwm1992 xwm1992 xwm1992 approved these changes

@lrhkobe lrhkobe lrhkobe approved these changes

@qqeasonchen qqeasonchen Awaiting requested review from qqeasonchen

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

1.10.0

Development

Successfully merging this pull request may close these issues.

[Feature] Rust sdk support CloudEvents

4 participants

@mxsm @xwm1992 @lrhkobe @pandaapo