You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@camel.apache.org by "Hadrian Zbarcea (JIRA)" <ji...@apache.org> on 2010/09/01 23:15:40 UTC
[jira] Commented: (CAMEL-3065) Netty - @deprecated sync option and
let it use the MEP to decide if its InOnly or InOut
[ https://issues.apache.org/activemq/browse/CAMEL-3065?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=61598#action_61598 ]
Hadrian Zbarcea commented on CAMEL-3065:
----------------------------------------
@Ashwin,
Thanks for the patch, it is indeed straightforward. Two comments though:
1. i think {{sendReply}} is equally confusing (in a different way) and my preference would be {{inOut}}. Since this is what happens anyway, the exchange will be InOut, it is imho clearer what the semantics of this option is.
2. You are now a committer, so please free to commit the code unless you have doubts in which case attaching a patch or adding a link to a git branch is ok. I doubt you had doubts in this case :).
> Netty - @deprecated sync option and let it use the MEP to decide if its InOnly or InOut
> ---------------------------------------------------------------------------------------
>
> Key: CAMEL-3065
> URL: https://issues.apache.org/activemq/browse/CAMEL-3065
> Project: Apache Camel
> Issue Type: Improvement
> Components: camel-netty
> Affects Versions: 2.4.0
> Reporter: Claus Ibsen
> Assignee: Ashwin Karpe
> Priority: Minor
> Fix For: 3.0.0
>
> Attachments: CAMEL-3065-fix.diff
>
>
> See nabble
> http://camel.465427.n5.nabble.com/netty-issue-tp2639835p2639835.html
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.