From 2795649e9747c8dc4030ec56866107f3af11cb8f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jorge=20Sol=C3=B3rzano?= Date: Mon, 22 Jul 2024 02:26:07 +0200 Subject: [PATCH] Register fields for reflection in kubernetes-client MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Jorge Solórzano --- .../deployment/KubernetesClientProcessor.java | 53 ++++--------------- 1 file changed, 10 insertions(+), 43 deletions(-) diff --git a/extensions/kubernetes-client/deployment/src/main/java/io/quarkus/kubernetes/client/deployment/KubernetesClientProcessor.java b/extensions/kubernetes-client/deployment/src/main/java/io/quarkus/kubernetes/client/deployment/KubernetesClientProcessor.java index dc8370dfd862a..13d7466e7465e 100644 --- a/extensions/kubernetes-client/deployment/src/main/java/io/quarkus/kubernetes/client/deployment/KubernetesClientProcessor.java +++ b/extensions/kubernetes-client/deployment/src/main/java/io/quarkus/kubernetes/client/deployment/KubernetesClientProcessor.java @@ -1,6 +1,5 @@ package io.quarkus.kubernetes.client.deployment; -import java.util.AbstractMap; import java.util.ArrayList; import java.util.Collection; import java.util.Collections; @@ -10,8 +9,6 @@ import java.util.function.Predicate; import java.util.stream.Collectors; -import org.jboss.jandex.AnnotationInstance; -import org.jboss.jandex.AnnotationTarget; import org.jboss.jandex.ClassInfo; import org.jboss.jandex.DotName; import org.jboss.jandex.IndexView; @@ -159,20 +156,16 @@ public void process(ApplicationIndexBuildItem applicationIndex, CombinedIndexBui Collection visitableBuilderImpls = fullIndex.getAllKnownImplementors(VISITABLE_BUILDER); // default sizes determined experimentally - these are only set in order to prevent continuous expansion of the array list - List withoutFieldsRegistration = new ArrayList<>( + List withFieldsRegistration = new ArrayList<>( kubernetesResourceImpls.size() + kubernetesResourceListImpls.size()); - List withFieldsRegistration = new ArrayList<>(2); List ignoreJsonDeserialization = new ArrayList<>( kubernetesResourceImpls.size() + kubernetesResourceListImpls.size()); populateReflectionRegistrationLists(kubernetesResourceImpls, watchedClasses, ignoreJsonDeserialization, - withoutFieldsRegistration, withFieldsRegistration); populateReflectionRegistrationLists(kubernetesResourceListImpls, watchedClasses, ignoreJsonDeserialization, - withoutFieldsRegistration, withFieldsRegistration); populateReflectionRegistrationLists(visitableBuilderImpls, watchedClasses, ignoreJsonDeserialization, - withoutFieldsRegistration, withFieldsRegistration); if (!withFieldsRegistration.isEmpty()) { @@ -180,11 +173,6 @@ public void process(ApplicationIndexBuildItem applicationIndex, CombinedIndexBui .builder(withFieldsRegistration.toArray(EMPTY_STRINGS_ARRAY)).weak(true).methods().fields() .build()); } - if (!withoutFieldsRegistration.isEmpty()) { - reflectiveClasses.produce(ReflectiveClassBuildItem - .builder(withoutFieldsRegistration.toArray(EMPTY_STRINGS_ARRAY)).weak(true).methods() - .build()); - } ignoredJsonDeserializationClasses.produce(new IgnoreJsonDeserializeClassBuildItem(ignoreJsonDeserialization)); @@ -233,9 +221,8 @@ public void process(ApplicationIndexBuildItem applicationIndex, CombinedIndexBui .sorted() .collect(Collectors.joining("\n")); log.debugv("Watched Classes:\n{0}", watchedClassNames); - List modelClasses = new ArrayList<>(withFieldsRegistration.size() + withoutFieldsRegistration.size()); + List modelClasses = new ArrayList<>(withFieldsRegistration.size()); modelClasses.addAll(withFieldsRegistration); - modelClasses.addAll(withoutFieldsRegistration); Collections.sort(modelClasses); log.debugv("Model Classes:\n{0}", String.join("\n", modelClasses)); } @@ -254,35 +241,15 @@ public void process(ApplicationIndexBuildItem applicationIndex, CombinedIndexBui private static void populateReflectionRegistrationLists(Collection kubernetesResourceImpls, Set watchedClasses, List ignoredJsonDeserializationClasses, - List withoutFieldsRegistration, List withFieldsRegistration) { - kubernetesResourceImpls - .stream() - .peek(c -> { - // we need to make sure that the Jackson extension does not try to fully register the model classes - // since we are going to register them weakly - ignoredJsonDeserializationClasses.add(c.name()); - }) - .filter(c -> !watchedClasses.contains(c.name())) - .map(c -> { - boolean registerFields = false; - List jsonFormatInstances = c.annotationsMap().get(JSON_FORMAT); - if (jsonFormatInstances != null) { - for (AnnotationInstance jsonFormatInstance : jsonFormatInstances) { - if (jsonFormatInstance.target().kind() == AnnotationTarget.Kind.FIELD) { - registerFields = true; - break; - } - } - } - return new AbstractMap.SimpleEntry<>(c.name(), registerFields); - }).forEach(e -> { - if (e.getValue()) { - withFieldsRegistration.add(e.getKey().toString()); - } else { - withoutFieldsRegistration.add(e.getKey().toString()); - } - }); + for (ClassInfo resource : kubernetesResourceImpls) { + // we need to make sure that the Jackson extension does not try to fully + // register the model classes since we are going to register them weakly + ignoredJsonDeserializationClasses.add(resource.name()); + if (!watchedClasses.contains(resource.name())) { + withFieldsRegistration.add(resource.name().toString()); + } + } } private void findWatchedClasses(final DotName implementedOrExtendedClass,