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] [Commented] (CAMEL-4475) Make API self contained
Date Thu, 22 Sep 2011 08:03:26 GMT

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

Claus Ibsen commented on CAMEL-4475:
------------------------------------

beforeWrap cannot be removed, as 3rd party depend up it. I think it was JBoss needing it back
then.

Yes keep the API in the model as much as possible. There is many ppl written 3rd party management/monitoring/tracing
tooling and whatnot where they access the model to get details etc.

Outside the model package, the NamedNode is fine to use. In fact its already there as part
of the OptionalIdentifiedDefinition.


> Make API self contained
> -----------------------
>
>                 Key: CAMEL-4475
>                 URL: https://issues.apache.org/jira/browse/CAMEL-4475
>             Project: Camel
>          Issue Type: Improvement
>          Components: camel-core
>    Affects Versions: 2.8.0
>            Reporter: Christian Schneider
>            Assignee: Christian Schneider
>             Fix For: 2.9.0
>
>         Attachments: camel-4475-1.patch, camel-4475-after.png, camel-4475-before.png
>
>
> The API, SPI packages of camel are already quite independent after the previous refactorings.
There are three classes in model that are still needed at the moment.
> ProcessorDefinition, RouteDefinition and FromDefiniton
> I propose to add the necessary methods to NamedNode to be able to reference most of the
functionality of the classes. For RouteDefinition I propose a special interface that adds
the ErrorHandler. For the rare cases where more functionality is needed a instanceof and a
cast can be used.
> I will attach a patch shortly that solves the issue and would be glad about some reviews.
> The patch can not be fully compatible but the changes are mostly in spi interfaces that
are probably rather rarely used.
> I will also add screenshots from structure 101 that show how much the architecture improves
after this patch.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message