EricWittmann edited a comment on pull request #5834:
URL: https://github.com/apache/camel/pull/5834#issuecomment-881475090


   OK that makes sense (sharing config between component and endpoints) - but I 
**think** this bug was found on a use-case where the configuration was on two 
different endpoints.  Does that suggest that the configuration is being 
cloned/copied even when it comes from a separate endpoint (vs from the 
component)?  For the record this is the first time I've looked at Camel, so I'm 
coming at it from a place of ignorance.  :)
   
   The bug was originally encountered by @shuawest while doing some work with 
Camel+Kafka+Apicurio Registry.
   
   **Update**: my concern is that, while this fix should address the problem 
@shuawest found in his test application, that may only work because his 
endpoints have the same set of `additionalProperties` (but with different 
values).  So the copy would happen, then the new `additionalProperties` would 
overwrite the cloned ones.  However, if the second endpoint had a subset of 
`additionalProperties`, then I think there might still be a problem.


-- 
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]


Reply via email to