Mailing-List: contact dev-help@camel.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@camel.apache.org
Date: Thu, 22 Sep 2011 08:03:26 +0000 (UTC)
From: "Claus Ibsen (JIRA)" <jira@apache.org>
To: dev@camel.apache.org
Message-ID: 
 <2104786545.1304.1316678606243.JavaMail.tomcat@hel.zones.apache.org>
In-Reply-To: 
 <419404395.51555.1316627050836.JavaMail.tomcat@hel.zones.apache.org>
Subject: [jira] [Commented] (CAMEL-4475) Make API self contained
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ 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