cmccabe commented on code in PR #17076:
URL: https://github.com/apache/kafka/pull/17076#discussion_r1759200568


##########
config/kraft/reconfig-server.properties:
##########
@@ -0,0 +1,132 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more

Review Comment:
   @jsancio we should use the same approach here in `server.properties` as in 
`broker.properties` and `controller.properties`
   
   As I commented above, the simplest approach is just to have a line for 
`controller.quorum.voters`, but have it be commented out, with an explanation 
about how to use the old style if desired. That would avoid the need for a 
separate file.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to