[GitHub] [kafka] philipnee commented on pull request #13956: MINOR: Remove thread leak from ConsumerBounceTest

2023-07-12 Thread via GitHub


philipnee commented on PR #13956:
URL: https://github.com/apache/kafka/pull/13956#issuecomment-1632936434

   Hey @divijvaidya - I actually tried to fix this probably a year ago, and my 
guess was some of the consumer didn't close cleanly and caused leak.


-- 
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: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [kafka] philipnee commented on pull request #13956: MINOR: Remove thread leak from ConsumerBounceTest

2023-07-11 Thread via GitHub


philipnee commented on PR #13956:
URL: https://github.com/apache/kafka/pull/13956#issuecomment-1631191677

   Hey @divijvaidya - I think the fixes make sense.  Did you verify the thread 
leak by checking the threat dump? I think I've used lsof command before.


-- 
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: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org