Skip to content

Comments

Added "challenges" related to this pattern#15886

Closed
azarboon wants to merge 1 commit intoapache:camel-4.8.xfrom
azarboon:patch-1
Closed

Added "challenges" related to this pattern#15886
azarboon wants to merge 1 commit intoapache:camel-4.8.xfrom
azarboon:patch-1

Conversation

@azarboon
Copy link

@azarboon azarboon commented Oct 9, 2024

I've added challenges related to using circuit breaker pattern and a workaround so readers can take more informed decision.

Target

  • I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

  • If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

  • I checked that each commit in the pull request has a meaningful subject line and body.
  • I have run mvn clean install -DskipTests locally from root folder and I have committed all auto-generated changes.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 9, 2024

🌟 Thank you for your contribution to the Apache Camel project! 🌟

🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run

  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot.

  • You can label PRs using build-all, build-dependents, skip-tests and test-dependents to fine-tune the checks executed by this PR.

  • Build and test logs are available in the Summary page. Only Apache Camel committers have access to the summary.

  • ⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@github-actions github-actions bot added the core label Oct 9, 2024
@davsclaus
Copy link
Contributor

We have similar thoughts like Apache Pekko team
https://github.com/apache/pekko/pull/1524/files

Copy link
Contributor

@davsclaus davsclaus left a comment

Choose a reason for hiding this comment

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

see prev comment

@azarboon azarboon closed this Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants