showuon commented on code in PR #20237:
URL: https://github.com/apache/kafka/pull/20237#discussion_r2250299240
##########
docs/ops.html:
##########
@@ -4498,9 +4498,16 @@ <h4 class="anchor-heading"><a
id="eligible_leader_replicas_upgrade" class="ancho
<p>Downgrades are safe to perform by setting
<code>eligible.leader.replicas.version=0</code>.</p>
<h4 class="anchor-heading"><a id="eligible_leader_replicas_tool"
class="anchor-link"></a><a href="#eligible_leader_replicas_tool">Tool</a></h4>
- <p>The ELR fields can be checked through the API DescribeTopicPartitions.
The admin client can fetch the ELR info by describing the topics.
- Also note that, if <code>min.insync.replicas</code> is updated for a
topic, the ELR field will be cleaned. If cluster default min ISR is updated,
- all the ELR fields will be cleaned.</p>
+ <p>The ELR fields can be checked through the API DescribeTopicPartitions.
The admin client can fetch the ELR info by describing the topics.</p>
+ <p>Note that when the ELR feature is enabled:</p>
+ <ul>
+ <li>The cluster-level <code>min.insync.replicas</code> config will be
added if there is not any. The value is the same as the static config in the
active controller.</li>
+ <li>The removal of <code>min.insync.replicas</code> config in
cluster-level is not allowed.</li>
+ <li>If the cluster-level <code>min.insync.replicas</code> is updated, even
if the value is unchanged, all the ELR state will be cleaned.</li>
+ <li>The previously set <code>min.insync.replicas</code> value at the
broker-level config will be removed. Please set in the cluster-level if
necessary.</li>
Review Comment:
Good point! In the `Notable changes in 4.1.0` section, added:
> The KIP-966 part 1: Eligible Leader Replicas(ELR) will be enabled by
default on the new clusters. After the ELR feature enabled, the previously set
<code>min.insync.replicas</code> value at the broker-level config will be
removed. Please set at the cluster-level if necessary. For further details,
please refer to <a
href="/{{version}}/documentation.html#eligible_leader_replicas">here</a>.
--
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]