chia7712 commented on PR #17998: URL: https://github.com/apache/kafka/pull/17998#issuecomment-2535339509
> Are RackAwareTest and AdminRackAwareTest testing any logic used when not running with ZooKeeper? It seems all the callers of the methods they check are from ZooKeeper logic. yes, `AdminRackAwareTest` can be removed directly, but `RackAwareAutoTopicCreationTest` is running under kraft now. Maybe we can remove `AdminRackAwareTest` and then merge `RackAwareTest` and `RackAwareAutoTopicCreationTest` into single class? -- 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
