Re: [PR] KAFKA-16789: Fix thread leak detection for event handler threads [kafka]

2024-05-19 Thread via GitHub
chia7712 merged PR #15984: URL: https://github.com/apache/kafka/pull/15984 -- 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:

Re: [PR] KAFKA-16789: Fix thread leak detection for event handler threads [kafka]

2024-05-18 Thread via GitHub
chia7712 commented on PR #15984: URL: https://github.com/apache/kafka/pull/15984#issuecomment-2118911871 I will merge this PR after QA gets work. -- 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

Re: [PR] KAFKA-16789: Fix thread leak detection for event handler threads [kafka]

2024-05-17 Thread via GitHub
chia7712 commented on code in PR #15984: URL: https://github.com/apache/kafka/pull/15984#discussion_r1605427011 ## core/src/test/scala/unit/kafka/utils/TestUtils.scala: ## @@ -1872,7 +1872,7 @@ object TestUtils extends Logging {

[PR] KAFKA-16789: Fix thread leak detection for event handler threads [kafka]

2024-05-17 Thread via GitHub
gaurav-narula opened a new pull request, #15984: URL: https://github.com/apache/kafka/pull/15984 Updates `TestUtils::verifyNoUnexpectedThreads` to check for all event queue threads instead of the incorrect QuorumController thread. Also updated `KafkaEventQueueTest` to use