chaokunyang commented on code in PR #2701:
URL: https://github.com/apache/fory/pull/2701#discussion_r2412975809


##########
java/fory-core/src/main/resources/META-INF/native-image/org.apache.fory/fory-core/reflection-config.json:
##########
@@ -5,5 +5,18 @@
     "allPublicConstructors": true,
     "allDeclaredMethods": true,
     "allPublicMethods": true
+  },
+  {
+    "name": "jdk.internal.reflect.ReflectionFactory",

Review Comment:
   Looks good to me overall. We can use Fory existing registry to discover 
user's registered types, and add necessary metadata to graalvm for those types, 
so at graalvm runtime, the object creation will work as expected. 
   
   The most critical graal-meta would be :
   - `proxy` related meta, so we can create JDK proxy at runtime without users 
to configure `proxy.json`
   - configure `unsafeAllocated` in reflect-meta.json
   



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to