lburgazzoli commented on a change in pull request #231: Add support for quarkus
provided event loop
URL: https://github.com/apache/camel-quarkus/pull/231#discussion_r331850515
##########
File path:
extensions/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/BuildProcessor.java
##########
@@ -66,7 +66,26 @@ CamelRegistryBuildItem registry(
RuntimeValue<Registry> registry = recorder.createRegistry();
- CamelSupport.services(applicationArchives).forEach(si -> {
+ CamelSupport.services(applicationArchives)
+ .filter(si -> {
+ //
+ // we need to filter out some services for being
registered as some
+ // extension configure them programmatically and if not
properly
+ // configured, they may cause troubles.
+ //
+ // As example, a reactive-executor can be provided by
adding a dep on
+ // camel-reactive-executor-vertx but if that service get
detected and
+ // initialized, it will create an instance of Vert.x
during static init
Review comment:
ok, found out that I was wrong so I probably had some bad code during the
development that leaded to the reactive executor being started in the wrong
phase.
I've updated the comment.
----------------------------------------------------------------
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]
With regards,
Apache Git Services
