[kafka] Upgrade Cruise Control to 3.0.4 for Java 17 and Java 21 support - #1403
[kafka] Upgrade Cruise Control to 3.0.4 for Java 17 and Java 21 support#1403jitender-goyal wants to merge 1 commit into
Conversation
jitender-goyal
commented
Sep 9, 2026
- Upgraded default CRUISE_CONTROL_VERSION to 3.0.4
- Added Java 21 and Gradle 8.5 compatibility
- Replaced hardcoded reporter jar path with wildcard cruise-control-metrics-reporter-*.jar
- Added --add-opens JVM options to KAFKA_OPTS for runtime reflection on Java 17+
- Preserved legacy build.gradle injection only for older 2.0.x branches
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
There was a problem hiding this comment.
Code Review
This pull request updates the default Cruise Control version to 3.0.4 to support Java 17 and 21, conditionally applies legacy build patches for older versions, and adds JVM reflection options for modern Java runtimes. The review feedback suggests ensuring backward compatibility with Java 8 by conditionally applying the --add-opens JVM options, broadening the legacy version check regex to support all 2.x versions, and using a more specific file selection instead of a wildcard to avoid copying auxiliary jars to the Kafka classpath.
- Upgraded default CRUISE_CONTROL_VERSION to 3.0.4 (Gradle 8.5, Scala 2.13, Kafka 3.5+) - Conditionally applied legacy build patches only for older 2.x releases - Filtered auxiliary jars (sources, javadoc, tests) when copying metrics reporter jar to Kafka classpath - Conditionally added --add-opens JVM options to KAFKA_OPTS for Java 9+ runtimes
641adc3 to
311dab3
Compare