dejan2609 commented on PR #23142:
URL: https://github.com/apache/kafka/pull/23142#issuecomment-5302274834

   Two new entries in `gradle/wrapper/gradle-wrapper.properties` are completely 
legit (they are introduced since Gradle version 9.5.0):
   ```
   retries=0
   retryBackOffMs=500
   ```
   Related links:
    - 
https://github.com/gradle/gradle/pull/36556/changes/3b76aaaf39ba5564c7989f17a40937a0e2bb2eaf
 Add retries and timeout for gradle download
    - 
https://github.com/gradle/gradle/pull/37261/changes/f9ea3b1df83ec86097e92e6aa70630352e52ff9d
 Rename 'retryTimeoutMs' to "retryBackOffMs"
    - 
https://docs.gradle.org/9.7.0/userguide/gradle_wrapper.html#sec:configuring_wrapper_retries
    - 
https://docs.gradle.org/9.7.0/javadoc/org/gradle/api/tasks/wrapper/Wrapper.html#getRetries()
   
   
    I did some testing:
     - Kafka distribution created using `./gradlew releaseTarGz`
     - cluster started, events written into topic, events being read -->> 
everything works fine :heavy_check_mark:   (test scenario: 
https://kafka.apache.org/quickstart)
    
   @serhiy-bzhezytskyy  :bulb: Hint: bump Gradle version to 9.7.1 that will be 
released in a next few days: 
https://github.com/gradle/gradle/milestone/352?closed=1 
   
   Overall: LGTM (non-binding) :white_check_mark:


-- 
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]

Reply via email to