You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by GitBox <gi...@apache.org> on 2022/01/06 18:04:20 UTC

[GitHub] [kafka] cmccabe commented on pull request #11593: KAFKA-13528: KRaft RegisterBroker should validate that the cluster ID matches

cmccabe commented on pull request #11593:
URL: https://github.com/apache/kafka/pull/11593#issuecomment-1006801028


   > Should we add an integration that shows the expected behavior on the broker side?
   
   It would be a difficult integration test to write, since the existing test framework sets up the same cluster ID for everything. To be honest, I don't think writing this would be the best use of our integration-test-writing time: converting many of the existing tests that do more realistic things would have a much higher ROI. Let's file a follow-up JIRA to write an integration test like this later and try to get this in.


-- 
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-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org