[ 
https://issues.apache.org/jira/browse/CAMEL-12638?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16554613#comment-16554613
 ] 

Lukasz commented on CAMEL-12638:
--------------------------------

{quote}So I just add a comment that we need treat FluentProducerTemplate as a 
Builder.
{quote}
[~njiang] I kind of feel like this is not enough... If we want to avoid further 
confusion and potential production issues (our case) I think the class should 
be either made thread-safe for all operations *or* **it should not be possible 
to use it as a _ProducerTemplate_ (and don't create a singleton of this class 
as part of spring-boot-starter).

> DefaultFluentProducerTemplate is not thread safe
> ------------------------------------------------
>
>                 Key: CAMEL-12638
>                 URL: https://issues.apache.org/jira/browse/CAMEL-12638
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-core
>    Affects Versions: 2.20.2
>            Reporter: Lukasz
>            Priority: Major
>         Attachments: image-2018-07-12-17-58-09-225.png
>
>
> I think we have rediscovered the CAMEL-10820 bug. A body of one request gets 
> replaced with a body of proceeding request, in our case we use *request()* 
> method instead of *asyncSend()*.
> We use camel together with spring-boot. Consider following code:
>  
> {code:java}
> @Service
> public class UseCamelService {
>    private FluentProducerTemplate producer;
>    @Autowired
>    public UseCamelService(FluentProducerTemplate producer) {
>       this.producer = producer;
>    }
>    public String getValueFromCamel(String body) {
>       return producer.to("route").withBody(body).request(String.class);
>    }
> }
> {code}
> If *UseCamelService.getValueFromCamel()* gets called from two different 
> threads it is possible for the latter one to override the body of the first 
> one.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to