lburgazzoli commented on a change in pull request #618: Fix #Build time 
FactoryFinders
URL: https://github.com/apache/camel-quarkus/pull/618#discussion_r367686101
 
 

 ##########
 File path: 
extensions/core/deployment/src/main/java/org/apache/camel/quarkus/core/deployment/BuildProcessor.java
 ##########
 @@ -128,9 +130,40 @@ void 
coreServiceFilter(BuildProducer<CamelServiceFilterBuildItem> filterBuildIte
         }
 
         @BuildStep
-        void 
serviceInfoTransformers(BuildProducer<CamelServiceInfoTransformerBuildItem> 
mapperBuildItems) {
-            mapperBuildItems.produce(
-                    new 
CamelServiceInfoTransformerBuildItem(CamelServiceInfoTransformers::configurer));
+        void coreServices(
+                ApplicationArchivesBuildItem archives,
+                BuildProducer<CamelServiceBuildItem> services) {
+
+            /* Registered only */
+            CamelSupport.propertyFiles(
+                    archives,
+                    "META-INF/services/org/apache/camel/component",
+                    "META-INF/services/org/apache/camel/language",
+                    "META-INF/services/org/apache/camel/dataformat")
 
 Review comment:
   The "tuning" here is about to carefully select which language/dataformat can 
be bound to the registry.
   
   As example camel creates components only once so every time you use a 
schema, then you'll end up using the same component instance. 
   
   For languages and dataformat that's not always true and this is because you 
can use the same dataformat multiple time in the same route with a different 
configuration.
   
   

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to