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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
