REASY commented on PR #13157: URL: https://github.com/apache/arrow/pull/13157#issuecomment-1133923500
@kou I think it makes sense to test separately for Java 8 and Java 11. Java 11 has new features, so the bytecode could look different. I'm getting the following error on Java 11 when run parquet reader locally: ``` WARNING: An illegal reflective access operation has occurred WARNING: Illegal reflective access by org.apache.arrow.memory.util.MemoryUtil (file:/home/%USER%/.cache/coursier/v1/https/repo1.maven.org/maven2/org/apache/arrow/arrow-memory-core/8.0.0/arrow-memory-core-8.0.0.jar) to field java.nio.Buffer.address WARNING: Please consider reporting this to the maintainers of org.apache.arrow.memory.util.MemoryUtil WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations WARNING: All illegal access operations will be denied in a future release ``` If arrow java supports Java 17, it has to be tested separately as well because of the following change in JDK 17: https://openjdk.java.net/jeps/403 > Strongly encapsulate all internal elements of the JDK, except for [critical internal APIs](https://openjdk.java.net/jeps/260#Description) such as sun.misc.Unsafe. It will no longer be possible to relax the strong encapsulation of internal elements via a single command-line option, as was possible in JDK 9 through JDK 16. -- 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]
