RongtongJin opened a new pull request, #10928: URL: https://github.com/apache/rocketmq/pull/10928
### Which Issue(s) This PR Fixes - Fixes #10927 ### Brief Description RocketMQ upgraded the Maven fastjson2 dependency to 2.0.63 in #10715. That version contains a JDK 8 cold-start regression which may throw: ```text LambdaConversionException: Invalid caller ``` The regression is fixed upstream in fastjson2 2.0.64: - https://github.com/alibaba/fastjson2/issues/7691 - https://github.com/alibaba/fastjson2/pull/7718 - https://github.com/alibaba/fastjson2/releases/tag/2.0.64 This change: - upgrades the Maven fastjson2 version from 2.0.63 to 2.0.64; - aligns the Bazel fastjson2 pin from 2.0.59 to 2.0.64; - does not add a RocketMQ-specific initialization-order workaround. ### How Did You Test This Change? Using Amazon Corretto `1.8.0_502-b07` in a fresh JVM, I ran: ```bash mvn -B -ntp -nsu -pl remoting -am \ -Djacoco.skip=true \ -Dtest=RemotingSerializableCompatTest \ -Dsurefire.failIfNoSpecifiedTests=false \ clean test ``` Results under the same environment and test command: | fastjson2 | Result | |-----------|--------| | 2.0.63 | FAILED — 2 tests, 1 error, `LambdaConversionException` | | 2.0.64 | PASSED — 2 tests, 0 failures/errors | Additional verification: - `mvn -pl remoting -am -DskipTests package`: passed; - Maven dependency tree for `common` and `remoting`: resolves fastjson2 2.0.64 only; - Maven and Bazel dependency pins are aligned at 2.0.64; - `git diff --check`: passed. -- 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]
