Avoid reference from api to builder (ErrorHandlerBuilder) and clean up model in
regard to error handler
-------------------------------------------------------------------------------------------------------
Key: CAMEL-4453
URL: https://issues.apache.org/jira/browse/CAMEL-4453
Project: Camel
Issue Type: Improvement
Affects Versions: 2.8.0
Reporter: Christian Schneider
Fix For: 2.9.0
Currently we have some lees than optimal designs in camel:
The api and spi reference ErrorHandlerbuilder which is a builder level
interface. As the builder needs access to almost anything this creates a big
cycle over all of camel.
We also use the ErrorHandlerBuilder in many places where no build is necessary.
In these cases rather a factory is needed.
The last thing is that we allow to change the ErrorHandlerBuilder in
ProcessDefintion. So it can be set everywhere though it really should only be
set on the start of a RouteDefinition.
So I propose to change several things:
- Create an interface ErrorHandlerFactory that only has the create method and
use it instead of ErrorHandlerBuilder where no builder is needed
- Remove ErrorHandlerBuilder in ProcessDefintion and only have it in
RouteDefintion
The first part should be made compatible using deprecation so people will not
be affected.
The second part is incompatible but I doubt many people use the errorhandler
outside the start of a Route. So I think we need to compatiblity measures.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
