chia7712 commented on code in PR #19104: URL: https://github.com/apache/kafka/pull/19104#discussion_r1980994112
########## docs/zk2kraft.html: ########## @@ -57,7 +57,7 @@ <h3 class="anchor-heading">Configurations</h3> <p> In KRaft mode, Kafka eliminates its dependency on ZooKeeper, and the control plane functionality is fully integrated into Kafka itself. The process roles are clearly separated: brokers handle data-related - requests, while the controllers (a.k.a., quorum controller) manages metadata-related requests. The controllers + requests, while the controllers (a.k.a., quorum controller) manages metadata-related requests. The controllers Review Comment: ditto ########## docs/zk2kraft.html: ########## @@ -32,8 +32,8 @@ <h3 class="anchor-heading">Configurations</h3> <li> <p> Removed password encoder-related configurations. These configurations were used in - ZooKeeper mode to define the key and backup key for encrypting sensitive data (e.g., passwords), - specify the algorithm and key generation method for password encryption (e.g., AES, RSA), and control + ZooKeeper mode to define the key and backup key for encrypting sensitive data (e.g., passwords), Review Comment: please revert those unrelated change -- 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