mimaison commented on code in PR #19216: URL: https://github.com/apache/kafka/pull/19216#discussion_r2003823244
########## core/src/test/scala/unit/kafka/log/LogCleanerManagerTest.scala: ########## @@ -1,886 +0,0 @@ -/** Review Comment: The approach we have taken so far is to not migrate the tests right now. If we rewrite both main and test at the same time it's much easier to introduce changes or bugs. So can we keep this class and just make the (hopefully small changes) to test the new Java LogCleanerManager? -- 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
