Merge pull request #50450 from gastaldi/grpc_reflection

Enable reflection on `GeneratedMessage` and `GeneratedMessage.Builder` implementations

Author: gastaldi

extensions/grpc-common/deployment/src/main/java/io/quarkus/grpc/common/deployment/GrpcCommonProcessor.java

@@ -20,8 +20,16 @@ public void configureNativeExecutable(CombinedIndexBuildItem combinedIndex,
             BuildProducer<ReflectiveClassBuildItem> reflectiveClass) {
 
         // we force the usage of the reflection invoker.
-        Collection<ClassInfo> messages = combinedIndex.getIndex()
+        Collection<ClassInfo> messagesV3 = combinedIndex.getIndex()
                 .getAllKnownSubclasses(GrpcDotNames.GENERATED_MESSAGE_V3);
+        for (ClassInfo message : messagesV3) {
+            reflectiveClass.produce(ReflectiveClassBuildItem.builder(message.name().toString()).methods()
+                    .fields().build());
+        }
+
+        // we force the usage of the reflection invoker.
+        Collection<ClassInfo> messages = combinedIndex.getIndex()
+                .getAllKnownSubclasses(GrpcDotNames.GENERATED_MESSAGE);
         for (ClassInfo message : messages) {
             reflectiveClass.produce(ReflectiveClassBuildItem.builder(message.name().toString()).methods()
                     .fields().build());
@@ -35,6 +43,12 @@ public void configureNativeExecutable(CombinedIndexBuildItem combinedIndex,
                     .fields().build());
         }
 
+        Collection<ClassInfo> buildersV3 = combinedIndex.getIndex().getAllKnownSubclasses(GrpcDotNames.MESSAGE_BUILDER_V3);
+        for (ClassInfo builder : buildersV3) {
+            reflectiveClass.produce(ReflectiveClassBuildItem.builder(builder.name().toString()).methods()
+                    .fields().build());
+        }
+
         Collection<ClassInfo> builders = combinedIndex.getIndex().getAllKnownSubclasses(GrpcDotNames.MESSAGE_BUILDER);
         for (ClassInfo builder : builders) {
             reflectiveClass.produce(ReflectiveClassBuildItem.builder(builder.name().toString()).methods()

extensions/grpc-common/deployment/src/main/java/io/quarkus/grpc/common/deployment/GrpcDotNames.java

@@ -2,15 +2,18 @@
 
 import org.jboss.jandex.DotName;
 
+import com.google.protobuf.GeneratedMessage;
 import com.google.protobuf.GeneratedMessageV3;
 import com.google.protobuf.ProtocolMessageEnum;
 
 import io.grpc.LoadBalancerProvider;
 import io.grpc.NameResolverProvider;
 
 public class GrpcDotNames {
-    static final DotName MESSAGE_BUILDER = DotName.createSimple(GeneratedMessageV3.Builder.class.getName());
+    static final DotName GENERATED_MESSAGE = DotName.createSimple(GeneratedMessage.class.getName());
     static final DotName GENERATED_MESSAGE_V3 = DotName.createSimple(GeneratedMessageV3.class.getName());
+    static final DotName MESSAGE_BUILDER = DotName.createSimple(GeneratedMessage.Builder.class.getName());
+    static final DotName MESSAGE_BUILDER_V3 = DotName.createSimple(GeneratedMessageV3.Builder.class.getName());
     static final DotName PROTOCOL_MESSAGE_ENUM = DotName.createSimple(ProtocolMessageEnum.class.getName());
     static final DotName NAME_RESOLVER_PROVIDER = DotName.createSimple(NameResolverProvider.class.getName());
     static final DotName LOAD_BALANCER_PROVIDER = DotName.createSimple(LoadBalancerProvider.class.getName());