aldettinger commented on a change in pull request #3316:
URL: https://github.com/apache/camel-quarkus/pull/3316#discussion_r755066212
##########
File path:
extensions-core/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/CamelContextProcessor.java
##########
@@ -165,17 +165,12 @@ public CamelRuntimeBuildItem runtime(
List<CamelRuntimeTaskBuildItem> runtimeTasks,
CamelConfig config) {
- for (CamelRoutesBuilderClassBuildItem item : routesBuilderClasses) {
- // don't add routes builders that are known by the container
- if (containerBeans.getClasses().contains(item.getDotName())) {
- continue;
- }
-
- recorder.addRoutes(context.getCamelContext(),
item.getDotName().toString());
Review comment:
I've spread the logic from the loop above in the code block below. So,
the logic is now executed only when config.routesDiscovery.enabled = true.
However, to my current understanding, when config.routesDiscovery.enabled =
false, then the routesBuilderClasses list is empty. As such, the logic in the
loop above was not executed when config.routesDiscovery.enabled = false.
--
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.
To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
