You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2019/08/28 05:24:00 UTC
[jira] [Commented] (KAFKA-8837) Several integration tests
incorrectly use verifyNonDaemonThreadsStatus
[ https://issues.apache.org/jira/browse/KAFKA-8837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16917425#comment-16917425 ]
ASF GitHub Bot commented on KAFKA-8837:
---------------------------------------
ijuma commented on pull request #7255: KAFKA-8837: KafkaMetricReporterClusterIdTest may not shutdown ZooKeeperTestHarness
URL: https://github.com/apache/kafka/pull/7255
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
> Several integration tests incorrectly use verifyNonDaemonThreadsStatus
> ----------------------------------------------------------------------
>
> 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
> Priority: Major
>
> @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.
> Second, this and several other tests do not actually create KafkaServer with the thread prefix, so this verification does not actually do anything (because threads are not prefixed with the test name).
> We should move `TestUtils.verifyNonDaemonThreadsStatus(this.getClass.getName)` to the end of `tearDown()`; and every test that calls verifyNonDaemonThreadsStatus should actually create KafkaServer with their test classname prefix.
> The reason I started looking into this is because one of the recent builds failed with many tests failing with "found unexpected threads...". So, fixing the above may at least help us to find the root cause, because this will increase the chance that a test failing with this error is the same test that leaks the thread. [https://builds.apache.org/job/kafka-trunk-jdk11/774/#showFailuresLink]
--
This message was sent by Atlassian Jira
(v8.3.2#803003)