[GitHub] [camel-quarkus] ppalaga edited a comment on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] [camel-quarkus] ppalaga edited a comment on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders

GitBox
ppalaga edited a comment on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders
URL: https://github.com/apache/camel-quarkus/pull/358#issuecomment-551883002
 
 
   @lburgazzoli I finally figured out why you expect the injection & co in an `@AppScoped` RoutesBuilder to work without the current fix: the RoutesCollector looks up the RoutesBuilders in Arc via RuntimeBeanRepository. I did not know that.
   
   Actually the current proposal (mostly) eliminates the duplicate route initialization problem we discussed in the chat. There are still two route builders seen by CamelMain, but now they are the same instance (bc. both come from Arc and Arc correctly returns the same one anytime it is queried). The same instance won't get initialized and added to the Context twice thanks to `RouteBuilder.checkInitialized()`.
   
   Let me think how to proceed here.

----------------------------------------------------------------
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:
[hidden email]


With regards,
Apache Git Services