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

Babak Vahdat commented on CAMEL-5880:
-------------------------------------

Steve,

In the meanwhile the changes of CAMEL-5979 has been applied which is the 
following commit:

http://svn.apache.org/viewvc?view=revision&revision=r1455750

So apparently the way we retrieve the {{messageSessionID}} has been changed on 
the consumer side through that ticket. Now with your patch in place I end up 
with OOM while doing {{mvn clean install}} on this module, but this was not the 
case before CAMEL-5979! Have you got any idea what the conflict is.

Also your included patch contains the change by {{MessagePredicate}} where 
reversing the SessionID has been removed but the comment {{Reverse session ID 
for reply}} is still there. So which one should be correct, the code or the 
comment?

Fell free to comment on CAMEL-5979 as well if you as *QuickFIX/J Guru* think 
the changes of CAMEL-5979 were not appropriate or if there would be a better 
way to do that.

                
> camel-quickfix: QuickfixjEndpoint should adhere the InOut exchange pattern if 
> you would ask for this MEP
> --------------------------------------------------------------------------------------------------------
>
>                 Key: CAMEL-5880
>                 URL: https://issues.apache.org/jira/browse/CAMEL-5880
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-quickfix
>    Affects Versions: 2.10.3
>            Reporter: Babak Vahdat
>            Priority: Minor
>         Attachments: requestreply-2.patch, requestreply.patch
>
>
> See 
> http://camel.465427.n5.nabble.com/camel-quickfix-RequestReplyExample-java-io-IOException-td5723769.html

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to