linghengqian opened a new pull request, #36295:
URL: https://github.com/apache/shardingsphere/pull/36295

   For #29052 .
   
   Changes proposed in this pull request:
     - Use Reachability Metadata Schema `v1.1.0` for all GraalVM Reachability 
Metadata. This is the successor to 
https://github.com/apache/shardingsphere/pull/36037 .
     - In Reachability Metadata Schema `v1.1.0`, the attributes 
`queryAllPublicMethods`, `queryAllDeclaredConstructors`, 
`queryAllDeclaredMethods`, and `queryAllPublicConstructors` are no longer 
meaningful.
     - The following 6 types of Config Schema `v1.0.0` files,
       - 
https://www.graalvm.org/docs/reference-manual/native-image/assets/reflect-config-schema-v1.0.0.json
       - 
https://www.graalvm.org/docs/reference-manual/native-image/assets/jni-config-schema-v1.0.0.json
       - 
https://www.graalvm.org/docs/reference-manual/native-image/assets/resource-config-schema-v1.0.0.json
       - 
https://www.graalvm.org/docs/reference-manual/native-image/assets/proxy-config-schema-v1.0.0.json
       - 
https://www.graalvm.org/docs/reference-manual/native-image/assets/serialization-config-schema-v1.0.0.json
       - 
https://www.graalvm.org/docs/reference-manual/native-image/assets/predefined-classes-config-schema-v1.0.0.json
     - Unifie into 
https://www.graalvm.org/docs/reference-manual/native-image/assets/reachability-metadata-schema-v1.1.0.json
 .
     - Unfortunately, a conflict between the Quarkus community and Oracle Labs 
over the JSON implementation in the Reachability Metadata Schema `v1.1.0` 
resulted in a bug in the parsing of `reason` nodes in `GraalVM CE for JDK 
24.0.2`. The `native-image` command-line tool now doesn't recognize `reason` 
nodes at all. An earlier investigation is available at 
https://github.com/quarkusio/quarkus/pull/42205#discussion_r2228202956 .
     - Due to the runtime checks introduced in 
https://github.com/oracle/graal/issues/7480 , JSON conversions for some 
third-party dependencies are inconsistent. Runtime checking of the XA 
DataSource class has been corrected.
   
   ---
   
   Before committing this PR, I'm sure that I have checked the following 
options:
   - [x] My code follows the [code of 
conduct](https://shardingsphere.apache.org/community/en/involved/conduct/code/) 
of this project.
   - [x] I have self-reviewed the commit code.
   - [x] I have (or in comment I request) added corresponding labels for the 
pull request.
   - [x] I have passed maven check locally : `./mvnw clean install -B -T1C 
-Dmaven.javadoc.skip -Dmaven.jacoco.skip -e`.
   - [x] I have made corresponding changes to the documentation.
   - [x] I have added corresponding unit tests for my changes.
   - [ ] I have updated the Release Notes of the current development version. 
For more details, see [Update Release 
Note](https://shardingsphere.apache.org/community/en/involved/contribute/contributor/)
   


-- 
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]

Reply via email to