davsclaus commented on PR #26819: URL: https://github.com/apache/camel/pull/26819#issuecomment-5812613188
Thanks for the PR. We are closing it as invalid: - `SimpleRegistry` is a public class. Changing its parent class from `LinkedHashMap` to `ConcurrentHashMap` is an API change. It also loses the order in which beans were added, and it rejects null keys and values that work today. - `CamelContext` does not support adding routes and beans from several threads at the same time, so a lock in `DefaultModel.addCustomBean` would only cover one of those paths. - The `b != null` guard handles a list that has already been corrupted instead of preventing the corruption. - There is no JIRA issue and no reproducer showing how Camel ends up in this state. If you run into this with a real deployment, please open a JIRA issue that describes how the routes are deployed. _Claude Code on behalf of Claus Ibsen_ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
