Skip to content

[ISSUE #4178]Retry using the storage capability is added. #4179

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

Closed
wants to merge 4 commits into from

Conversation

haijunwang
Copy link

@haijunwang haijunwang commented Jul 3, 2023

Fixes #4178 .

Motivation

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

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
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Welcome to the Apache EventMesh community!!
This is your first PR in our project. We're very excited to have you onboard contributing. Your contributions are greatly appreciated!

Please make sure that the changes are covered by tests.
We will be here shortly.
Let us know if you need any help!

Want to get closer to the community?

WeChat Assistant WeChat Public Account Slack
Join Slack Chat

Mailing Lists:

Name Description Subscribe Unsubscribe Archive
Users User support and questions mailing list Subscribe Unsubscribe Mail Archives
Development Development related discussions Subscribe Unsubscribe Mail Archives
Commits All commits to repositories Subscribe Unsubscribe Mail Archives
Issues Issues or PRs comments and reviews Subscribe Unsubscribe Mail Archives

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.

@haijunwang Please create issue first and link this pr to it.

@haijunwang
Copy link
Author

@haijunwang Please create issue first and link this pr to it.
#4178

@xwm1992 xwm1992 changed the title Retry using the storage capability is added. [ISSUE #4178]Retry using the storage capability is added. Jul 5, 2023
@Alonexc
Copy link
Contributor

Alonexc commented Jul 5, 2023

please check checkstyle

@pandaapo
Copy link
Member

pandaapo commented Nov 14, 2023

This issue was fixed in PR #4544 by decoupling the storage medium on which the retry depends from EventMesh. Sincerely thank you for raising this issue and for your work in this PR. Welcome to experience the repair solution PR #4544.

该issue以将重试依赖的存储介质和EventMesh解耦的方式,在PR #4544中被修复。真诚地感谢您提出该issue以及在该PR中所做的工作。欢迎您体验PR #4544的修复方案。

@pandaapo pandaapo closed this Nov 14, 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.

[Bug] When there are a large number of errors in the HTTP target, RocketMQ consumption is slow
5 participants