[GitHub] [camel-quarkus] aldettinger commented on a diff in pull request #4088: JCache native mode

[GitBox <gi...@apache.org>]

aldettinger commented on code in PR #4088:
URL: https://github.com/apache/camel-quarkus/pull/4088#discussion_r970660322


##########
extensions/jcache/deployment/src/main/java/org/apache/camel/quarkus/component/jcache/deployment/JcacheProcessor.java:
##########
@@ -34,13 +38,20 @@ FeatureBuildItem feature() {
         return new FeatureBuildItem(FEATURE);
     }
 
-    /**
-     * Remove this once this extension starts supporting the native mode.
-     */
-    @BuildStep(onlyIf = NativeBuild.class)
-    @Record(value = ExecutionTime.RUNTIME_INIT)
-    void warnJvmInNative(JvmOnlyRecorder recorder) {
-        JvmOnlyRecorder.warnJvmInNative(LOG, FEATURE); // warn at build time
-        recorder.warnJvmInNative(FEATURE); // warn at runtime
+    @BuildStep
+    void nativeResources(BuildProducer<ServiceProviderBuildItem> services,
+            BuildProducer<NativeImageResourceBuildItem> nativeResources) {

Review Comment:
   Indeed, the `nativeResources` method param looks not to be used. Also, I wonder whether methods like `ServiceProviderBuildItem.allProvider*` could help 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.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org