jokerzsd opened a new pull request, #11155: URL: https://github.com/apache/rocketmq/pull/11155
## What is the purpose of the change Two independent fixes from issue #10978: 1. **Nullable getters**: annotate the seven `Message` getters that return null under normal conditions — `getTags()`, `getKeys()`, `getProperty()`, `getUserProperty()`, `getBuyerId()`, `getProperties()`, `getTransactionId()` — with `javax.annotation.Nullable`. Pure metadata, no behavioral change; lets Java IDEs and Kotlin callers detect the nullability. 2. **kotlin-stdlib transitive leak**: exclude `com.squareup.okhttp3:okhttp` from the `opentelemetry-exporter-otlp` dependency in `common/pom.xml`. The unused HTTP sender (RocketMQ uses the gRPC exporter) pulled `kotlin-stdlib` into every pure-Java consumer. This mirrors the kotlin exclusions already applied to the `okio-jvm` edge in the root pom. ## Brief changelog - `common/src/main/java/org/apache/rocketmq/common/message/Message.java`: add `@Nullable` to seven getters. - `common/pom.xml`: add okhttp exclusion to `opentelemetry-exporter-otlp`. ## Verifying this change - `mvn -f common/pom.xml compile` — BUILD SUCCESS (checkstyle + spotbugs pass). - `mvn -f common/pom.xml dependency:tree -Dincludes=org.jetbrains.kotlin` — no kotlin artifacts remain in the tree. Closes #10978 -- 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]
