Re: [PR] MINOR: Simplify Gradle upgrade [kafka]

2025-05-22 Thread via GitHub
fvaleri commented on PR #19780: URL: https://github.com/apache/kafka/pull/19780#issuecomment-2903363249 > Sorry, I don't know about the gradlewAll deprecation note. Could you tell me more? There is a deprecation note inside the file in the root directory, but I see this was recently

Re: [PR] MINOR: Simplify Gradle upgrade [kafka]

2025-05-21 Thread via GitHub
fvaleri commented on PR #19780: URL: https://github.com/apache/kafka/pull/19780#issuecomment-2898049319 cc @showuon I was also wondering if we still need to keep `gradleAll` given the deprecation note, and why are we using `gradle-8.14-all.zip` instead of `gradle-8.14-bin.zip`?

[PR] MINOR: Simplify Gradle upgrade [kafka]

2025-05-21 Thread via GitHub
fvaleri opened a new pull request, #19780: URL: https://github.com/apache/kafka/pull/19780 Today, we have to change multiple files when upgrading Gradle: - gradle/dependencies.gradle - gradle/wrapper/gradle-wrapper.properties - gradlew By using `$gradle.gradleVersion` and