cmccabe commented on pull request #11066: URL: https://github.com/apache/kafka/pull/11066#issuecomment-881054973
Can we do the integration test approach, but have it be part of `RaftClusterTest.scala`? It seems weird to have a new test file just for this one test. We could also check the metrics as part of an existing test in RaftClusterTest.scala. Perhaps as part of the create + close test? That would avoid consuming more time. Also, can we rename RaftClusterTest.scala => KRaftClusterTest.scala ? That's been bothering me for a while. :) -- 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
