Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

CAMEL-21614: camel-core - Prevent cache change miss on queue swap - 4.8 #16820

Conversation

essobedo
Copy link
Contributor

Fix for https://issues.apache.org/jira/browse/CAMEL-21614 (4.8)

Motivation

In some specific use cases, the eviction of the entries never ends

Modifications:

  • Improve the way to swap the queue of change to avoid the race condition that could cause cache changes miss by leveraging a ReadWriteLock to only prevent change additions during the swap

Copy link
Contributor

🌟 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.

@essobedo essobedo changed the title CAMEL-21614: camel-core - Prevent cache change miss on queue swap for 4.8 CAMEL-21614: camel-core - Prevent cache change miss on queue swap - 4.8 Jan 15, 2025
@oscerd
Copy link
Contributor

oscerd commented Jan 15, 2025

Don't worry about the failure. It's because it must be rebuilt with the new SNAPSHOT version.

@essobedo
Copy link
Contributor Author

I know, I'm currently doing the Regen on the branch, thx anyway for the info

@essobedo essobedo force-pushed the essobedo/CAMEL-21614/prevent-cache-change-miss-on-swap-4.8 branch from 972f95c to 65c0f56 Compare January 15, 2025 08:50
@essobedo
Copy link
Contributor Author

Regen done, it should be good now

@essobedo essobedo merged commit 4226f0f into camel-4.8.x Jan 15, 2025
4 checks passed
@essobedo essobedo deleted the essobedo/CAMEL-21614/prevent-cache-change-miss-on-swap-4.8 branch January 15, 2025 09:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants