soarez commented on PR #15617: URL: https://github.com/apache/kafka/pull/15617#issuecomment-2026228502
Olá Filipe. Thanks for your contribution. `BrokerServer` shouldn't aim to follow the structure or interface in `KafkaServer`. What those two are meant to have in common is defined in `KafkaBroker`. So on its own, I don't think this change makes much sense. If you have a test that's going to make use of this, perhaps you could bundle those two changes in the same PR? -- 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