You are viewing a plain text version of this content. The canonical link for it is here.

Posted to jira@kafka.apache.org by "Chia-Ping Tsai (Jira)" <ji...@apache.org> on 2024/07/22 17:49:00 UTC

[jira] [Resolved] (KAFKA-17159) Make sure kafka-cluster-test-kit-executor get down when closing KafkaClusterTestKit

     [ https://issues.apache.org/jira/browse/KAFKA-17159?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Chia-Ping Tsai resolved KAFKA-17159.
------------------------------------
    Fix Version/s: 3.9.0
       Resolution: Fixed

> Make sure kafka-cluster-test-kit-executor get down when closing KafkaClusterTestKit
> -----------------------------------------------------------------------------------
>
>                 Key: KAFKA-17159
>                 URL: https://issues.apache.org/jira/browse/KAFKA-17159
>             Project: Kafka
>          Issue Type: Sub-task
>            Reporter: PoAn Yang
>            Assignee: PoAn Yang
>            Priority: Minor
>             Fix For: 3.9.0
>
>
> When trying to remove `TestUtils.waitForCondition` in [https://github.com/apache/kafka/pull/16499], kraft may have thread leak.
> We can use ThreadUtils#shutdownExecutorServiceQuietly [0] to replace the block in KafkaClusterTestKie#close [1].
>  
> [0] [https://github.com/apache/kafka/blob/f595802cc752ed01dc74e9ab932209fe25a9d10b/clients/src/main/java/org/apache/kafka/common/utils/ThreadUtils.java#L71-L92]
> [1] [https://github.com/apache/kafka/blob/f595802cc752ed01dc74e9ab932209fe25a9d10b/core/src/test/java/kafka/testkit/KafkaClusterTestKit.java#L640-L643]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)