TaiJuWu commented on code in PR #16957:
URL: https://github.com/apache/kafka/pull/16957#discussion_r1728049863


##########
core/src/test/java/kafka/test/annotation/ClusterTest.java:
##########
@@ -34,6 +35,7 @@
 @Target({METHOD})
 @Retention(RUNTIME)
 @TestTemplate
+@Timeout(30)

Review Comment:
   > Hello @TaiJuWu, In this PR, we're setting a 30-second timeout limit for 
all new test infrastructure. We’ve already identified and fixed test cases 
exceeding this limit in a [previous 
PR](https://github.com/apache/kafka/pull/16796) and addressed these issues in a 
follow-up [PR](https://github.com/apache/kafka/pull/16810). Moving forward, 
developers should ensure their tests complete within this 30-second window to 
avoid prolonged execution times. WDYT?
   
   Got it. Sound great. Thanks for your explain.



-- 
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: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to