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 #405 Fix SubscriptionType typo error #522

Merged
merged 19 commits into from
Sep 15, 2021

Conversation

jinrongluo
Copy link
Contributor

Fixes ISSUE#405

Motivation

Fix SubscriptionType field typo error in SubscriptionItem Class

Modifications

Describe the modifications you've done.

Documentation

  • Does this pull request introduce a new feature? 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

@codecov-commenter
Copy link

Codecov Report

Merging #522 (1d7c059) into develop (bd8d736) will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             develop     #522   +/-   ##
==========================================
  Coverage      10.42%   10.42%           
  Complexity       328      328           
==========================================
  Files            241      241           
  Lines          11737    11737           
  Branches        1001     1001           
==========================================
  Hits            1224     1224           
  Misses         10411    10411           
  Partials         102      102           
Impacted Files Coverage Δ
...he/eventmesh/common/protocol/SubscriptionItem.java 0.00% <0.00%> (ø)
...he/eventmesh/common/protocol/SubscriptionType.java 0.00% <0.00%> (ø)
.../core/protocol/http/push/AsyncHTTPPushRequest.java 0.00% <0.00%> (ø)
...rotocol/tcp/client/session/push/SessionPusher.java 0.00% <0.00%> (ø)
...client/session/push/retry/EventMeshTcpRetryer.java 0.00% <0.00%> (ø)
...ache/eventmesh/client/tcp/common/MessageUtils.java 0.00% <0.00%> (ø)
...ntmesh/client/tcp/impl/DefaultEventMeshClient.java 0.00% <0.00%> (ø)
...eventmesh/client/tcp/impl/SimpleSubClientImpl.java 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bd8d736...1d7c059. Read the comment docs.

Copy link
Member

@ruanwenjun ruanwenjun left a comment

Choose a reason for hiding this comment

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

LGTM

@xwm1992 xwm1992 merged commit 84b2f4d into apache:develop Sep 15, 2021
@qqeasonchen
Copy link
Contributor

LGTM

@jinrongluo jinrongluo deleted the develop-bug-fix branch September 17, 2021 13:25
xwm1992 pushed a commit to xwm1992/EventMesh that referenced this pull request Dec 27, 2021
* [Issue apache#337] Fix HttpSubscriber startup issue

* [Issue apache#337] test commit

* [Issue apache#337] revert test commit

* [Issue apache#337] Enhance Http Demo Subscriber by using ExecutorService, CountDownLatch and PreDestroy hook

* [Issue apache#337] Enhance Http Demo Subscriber by using ExecutorService, CountDownLatch and PreDestroy hook

* [Issue apache#337] Address code review comment for Subscriber Demo App

* Fix subscriptionType typo

Co-authored-by: j00441484 <[email protected]>
xwm1992 pushed a commit that referenced this pull request Aug 4, 2022
* [Issue #337] Fix HttpSubscriber startup issue

* [Issue #337] test commit

* [Issue #337] revert test commit

* [Issue #337] Enhance Http Demo Subscriber by using ExecutorService, CountDownLatch and PreDestroy hook

* [Issue #337] Enhance Http Demo Subscriber by using ExecutorService, CountDownLatch and PreDestroy hook

* [Issue #337] Address code review comment for Subscriber Demo App

* Fix subscriptionType typo

Co-authored-by: j00441484 <[email protected]>
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.

5 participants