ppalaga commented on issue #1840:
URL: https://github.com/apache/camel-quarkus/issues/1840#issuecomment-703591612


   camel-lra instantiates `LRASagaRoutes` programmatically, providing a 
`LRASagaService` to the `LRASagaRoutes(LRASagaService sagaService)` 
constructor. I think it is really not a good idea to let Camel Quarkus Core to 
instantiate it. Of course unless we redesign camel-lra to suit better what 
Camel Quarkus expects. Yes, we will perhaps need to change camel-lra anyway, 
when porting to native. The solution with `RoutesBuilderClassExcludeBuildItem` 
only aims at improving the UX of camel-lra in JVM mode, so that it is there for 
Camel K.
   
   I was also thinking of excluding all RoutesBuilder classes which do not have 
a no-args constructor. I do not think it is a good idea because RoutesBuilders 
may lack no-args constructors for other reasons too - e.g. by mistake.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to