Skip to content

[CAMEL-20829] Streamline ServicePool synchronization#14312

Merged
gnodet merged 1 commit into
apache:mainfrom
gnodet:fix-servicepool-synchronization
Jun 4, 2024
Merged

[CAMEL-20829] Streamline ServicePool synchronization#14312
gnodet merged 1 commit into
apache:mainfrom
gnodet:fix-servicepool-synchronization

Conversation

@gnodet
Copy link
Copy Markdown
Contributor

@gnodet gnodet commented Jun 3, 2024

As we've seen in recent PRs, ServicePool is over-synchronized, as when upgrading the cache
to use concurrent data structures, the old mechanism using a look was left in place.
Also, the lock when stopping was caused by trying to stop the service before removing
it from the queue (that was fix in a recent commit).

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 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.

@davsclaus
Copy link
Copy Markdown
Contributor

davsclaus commented Jun 4, 2024

/component-test camel-ftp

Result ✅ The tests passed successfully

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 4, 2024

🤖 The Apache Camel test robot will run the tests for you 👍

@gnodet gnodet changed the title Streamline ServicePool synchronization [CAMEL-20829] Streamline ServicePool synchronization Jun 4, 2024
@gnodet gnodet merged commit 3fe53e0 into apache:main Jun 4, 2024
@gnodet gnodet deleted the fix-servicepool-synchronization branch January 27, 2026 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants