janchilling commented on PR #19889: URL: https://github.com/apache/kafka/pull/19889#issuecomment-2940839909
Hi @cadonna , I’ve added a test case that verifies both the starting and shutdown behavior of the `StateUpdater` thread—since we shouldn't leave the thread running after the test. To perform the verification, I used `DefaultStateUpdater` directly in order to access the `isRunning()` method, as the existing mock `StateUpdater` doesn't expose it. This meant writing a more customized test rather than reusing the existing setup methods. I believe this addresses all the requested changes, but please let me know if there’s anything else you'd like me to update. -- 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
