davsclaus commented on pull request #5543:
URL: https://github.com/apache/camel/pull/5543#issuecomment-839542763


   Okay I did a bit of test, and we have tests from camel-core that would 
create an exchange from another endpoint (such as file endpoint) and then send 
to direct endpoint via a producer template to be routed.
   
   In that situation then your patch will override the existing from endpoint 
(the file endpoint) with the route endpoint (direct), which would be wrong IMHO.
   
   So you should check for null, before setting the endpoint


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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to