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

Reply via email to