[
https://issues.apache.org/jira/browse/KAFKA-17151?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
PoAn Yang updated KAFKA-17151:
------------------------------
Description:
discussion: [https://github.com/apache/kafka/pull/16499#discussion_r1679676258]
Those tests, which are using ClusterTestExtensions, should make sure the
threads created by testing get done indeed.
BTW, we need to ensure removing `waitForCondition` won't bring a bunch of
failed tests, and hence please feel free to add sub-task if you notice the
thread which can cause failure after removing the `waitForCondition`.
The PR for this ticket will also include KAFKA-17187 and KAFKA-17187, so we can
make sure there is no leak thread after applying these fix.
was:
discussion: https://github.com/apache/kafka/pull/16499#discussion_r1679676258
Those tests, which are using ClusterTestExtensions, should make sure the
threads created by testing get done indeed.
BTW, we need to ensure removing `waitForCondition` won't bring a bunch of
failed tests, and hence please feel free to add sub-task if you notice the
thread which can cause failure after removing the `waitForCondition`
> Remove waitForCondition when detecting thread leak
> --------------------------------------------------
>
> Key: KAFKA-17151
> URL: https://issues.apache.org/jira/browse/KAFKA-17151
> Project: Kafka
> Issue Type: Improvement
> Reporter: Chia-Ping Tsai
> Assignee: PoAn Yang
> Priority: Minor
>
> discussion:
> [https://github.com/apache/kafka/pull/16499#discussion_r1679676258]
> Those tests, which are using ClusterTestExtensions, should make sure the
> threads created by testing get done indeed.
> BTW, we need to ensure removing `waitForCondition` won't bring a bunch of
> failed tests, and hence please feel free to add sub-task if you notice the
> thread which can cause failure after removing the `waitForCondition`.
>
> The PR for this ticket will also include KAFKA-17187 and KAFKA-17187, so we
> can make sure there is no leak thread after applying these fix.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
