Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

pan3793
Copy link
Member

@pan3793 pan3793 commented Aug 7, 2025

The current ASM 9.7 does not support JDK 24 and 25 bytecode.

...
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-shade-plugin:3.6.0:shade (default) on project kyuubi-spark-sql-engine_2.12: Error creating shaded jar: Problem shading JAR /Users/chengpan/.m2/repository/net/bytebuddy/byte-buddy/1.17.6/byte-buddy-1.17.6.jar entry META-INF/versions/24/net/bytebuddy/jar/asmjdkbridge/JdkClassWriter$1.class: java.lang.IllegalArgumentException: Unsupported class file major version 68 -> [Help 1]
...

The current ASM 9.7 does not support JDK 24 and 25 bytecode.

```
...
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-shade-plugin:3.6.0:shade (default) on project kyuubi-spark-sql-engine_2.12: Error creating shaded jar: Problem shading JAR /Users/chengpan/.m2/repository/net/bytebuddy/byte-buddy/1.17.6/byte-buddy-1.17.6.jar entry META-INF/versions/24/net/bytebuddy/jar/asmjdkbridge/JdkClassWriter$1.class: java.lang.IllegalArgumentException: Unsupported class file major version 68 -> [Help 1]
...
```
@pan3793
Copy link
Member Author

pan3793 commented Aug 7, 2025

the failed UT does not look related to this change, can any committer review this PR?

@olamy olamy added the dependencies Pull requests that update a dependency file label Aug 7, 2025
@olamy olamy self-assigned this Aug 7, 2025
@olamy olamy merged commit d008516 into apache:master Aug 7, 2025
36 of 40 checks passed
@github-actions github-actions bot added this to the 3.6.1 milestone Aug 7, 2025
@ludoch
Copy link

ludoch commented Sep 8, 2025

jdk25 release is next week, but not sure this change was pushed to a new release of the plugin...

@olamy
Copy link
Member

olamy commented Sep 8, 2025

jdk25 release is next week, but not sure this change was pushed to a new release of the plugin...

looks not. I will have a look to cut release.
Even if it's still possible to override the asm dependency in your pom, I agree it's not really convenient.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants