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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
