unknowntpo opened a new pull request, #23604: URL: https://github.com/apache/kafka/pull/23604
`testRemoteReadFetchDataInfo` creates its own `RemoteLogManager`, which shadows the one created in `setUp`, so `tearDown` never closes it. Its RLM tasks keep running after the test finishes and can interfere with the tests that follow, for example by re-registering broker topic metrics through `BrokerTopicMetrics.GaugeWrapper`. Use try-with-resources so the manager is closed when the test ends. This came up while working on a follow-up to #22873: once `listRemoteLogSegments` has a default stub in `setUp`, the leaked manager's tasks stop failing early and `testRemoteLogManagerTasksAvgIdlePercentAndMetadataCountMetrics` fails when the class runs as a whole. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
