rondagostino commented on a change in pull request #10199:
URL: https://github.com/apache/kafka/pull/10199#discussion_r583226737



##########
File path: core/src/main/scala/kafka/server/KafkaConfig.scala
##########
@@ -675,6 +676,7 @@ object KafkaConfig {
     "KIP-500. If it is not set, the metadata log is placed in the first log 
directory from log.dirs."
   val ControllerListenerNamesDoc = "A comma-separated list of the names of the 
listeners used by the KIP-500 controller. This is required " +
     "if this process is a KIP-500 controller. The ZK-based controller will not 
use this configuration."
+  val SaslMechanismControllerProtocolDoc = "SASL mechanism used for 
broker-to-raft-controller communication. Default is GSSAPI."

Review comment:
       Should probably say what KIP-631 says, which is "SASL mechanism used for 
communication with controllers. 
    Default is GSSAPI." It is used by Raft controllers to talk to other 
controllers, after all (assuming we are using SASL for inter-controller 
communication).  Will fix.




----------------------------------------------------------------
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.

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


Reply via email to