Skip to content

Conversation

@hhuang1231
Copy link
Contributor

@hhuang1231 hhuang1231 commented Nov 27, 2023

Fixes #4581.

Motivation

#4581

Modifications

avoid http demo and grpc demo rely on tcp runtime

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

@pandaapo pandaapo left a comment

Choose a reason for hiding this comment

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

Does org.apache.eventmesh.grpc.sub.app.SpringBootDemoApplication also need to be processed similarly?

@hhuang1231
Copy link
Contributor Author

Does org.apache.eventmesh.grpc.sub.app.SpringBootDemoApplication also need to be processed similarly?

yes,I missed it.

@codecov
Copy link

codecov bot commented Nov 27, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (4e72b3c) 16.95% compared to head (fb4ec6e) 16.95%.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #4583   +/-   ##
=========================================
  Coverage     16.95%   16.95%           
  Complexity     1678     1678           
=========================================
  Files           781      781           
  Lines         29167    29167           
  Branches       2510     2510           
=========================================
  Hits           4945     4945           
  Misses        23761    23761           
  Partials        461      461           

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

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.

LGTM

@mxsm mxsm merged commit 6139147 into apache:master Nov 27, 2023
wizardzhang pushed a commit to wizardzhang/eventmesh that referenced this pull request Nov 28, 2023
* fix : http demo rely on tcp runtime

* fix : grpc demo rely on tcp runtime
@hhuang1231 hhuang1231 deleted the examples branch November 28, 2023 15:24
xuhongjia pushed a commit to Deckers-Ohana/eventmesh that referenced this pull request Mar 13, 2025
* fix : http demo rely on tcp runtime

* fix : grpc demo rely on tcp runtime
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.

[Question] When executing the code in examples, is it necessary to enable the runtime of the tcp protocol?

3 participants