camel-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Claus Ibsen (JIRA)" <j...@apache.org>
Subject [jira] Issue Comment Edited: (CAMEL-1078) Do not copy Exchanges at each step to improve performance
Date Fri, 03 Jul 2009 05:52:07 GMT

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

Claus Ibsen edited comment on CAMEL-1078 at 7/2/09 10:50 PM:
-------------------------------------------------------------

Exchange specialized classes to be removed:
[DONE] - BeanExchange
[DONE] - CxfExchange
[DONE] - DummyExchange
[DONE] - FileExchange
[DONE] - HttpExchange
[DONE] - IrcExchange
[DONE] - JhcExchange
[DONE] - JmsExchange
[DONE] - JMXExchange
[DONE] - MailExchange
[DONE] - MinaExchange
[DONE] - MyExchange
[DONE] - QuartzExchange
[DONE] - RemoteFileExchange
[DONE] - SpringIntegrationExchange
TraceEventExchange
[DONE] - XmppExchange

And some hard ones:
[DONE] - GenericFileExchange
[DONE] - RemoteFileExchange


      was (Author: hadrian):
    Exchange specialized classes to be removed:
[DONE] - BeanExchange
[DONE] - CxfExchange
[DONE] - DummyExchange
[DONE] - FileExchange
[DONE] - HttpExchange
[DONE] - IrcExchange
[DONE] - JhcExchange
[DONE] - JmsExchange
[DONE] - JMXExchange
[DONE] - MailExchange
[DONE] - MinaExchange
[DONE] - MyExchange
[DONE] - QuartzExchange
[DONE] - RemoteFileExchange
[DONE] - SpringIntegrationExchange
TraceEventExchange
[DONE] - XmppExchange

And some hard ones:
GenericFileExchange
[DONE] - RemoteFileExchange

  
> Do not copy Exchanges at each step to improve performance
> ---------------------------------------------------------
>
>                 Key: CAMEL-1078
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-1078
>             Project: Apache Camel
>          Issue Type: Improvement
>          Components: camel-core
>    Affects Versions: 1.5.0
>            Reporter: Hadrian Zbarcea
>            Assignee: Hadrian Zbarcea
>            Priority: Critical
>             Fix For: 2.0.0
>
>
> The code in DefaultEndpoint.createExchange(Exchange exchange), calls on getExchangeType()
to figure out if a new copy of Exchange needs be created.  The code relies though on the generic
argument of the Endpoint type to figure out the actual Exchange type.  With the generics removed
part of CAMEL-872, getExchangeType() always returns null, forcing a copy every time.
> There is no real need to extend Exchange, so the proposal for this to make the Exchange
final and move the component specific bits, if any,  in the Message.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message