Anna Povzner created KAFKA-8837:
-----------------------------------
Summary: KafkaMetricReporterClusterIdTest may not shutdown
ZooKeeperTestHarness
Key: KAFKA-8837
URL: https://issues.apache.org/jira/browse/KAFKA-8837
Project: Kafka
Issue Type: Bug
Components: core
Reporter: Anna Povzner
Assignee: Anastasia Vela
@After method in KafkaMetricReporterClusterIdTest calls
`TestUtils.verifyNonDaemonThreadsStatus(this.getClass.getName)` before it calls
tearDown on ZooKeeperTestHarness (which shut downs ZK and zk client). If
verifyNonDaemonThreadsStatus asserts, the rest of the resources will not get
cleaned up.
We should move `TestUtils.verifyNonDaemonThreadsStatus(this.getClass.getName)`
to the end of `tearDown()`. However, would also be good to prevent people using
this method in tear down similarly in the future. Maybe just adding a comment
would help here.
--
This message was sent by Atlassian Jira
(v8.3.2#803003)
