This is an automated email from the ASF dual-hosted git repository.
mimaison pushed a commit to branch 4.0
in repository https://gitbox.apache.org/repos/asf/kafka.git
The following commit(s) were added to refs/heads/4.0 by this push:
new 9e80d7ed564 KAFKA-18363: Remove ZooKeeper mentiosn in broker configs
(#18346)
9e80d7ed564 is described below
commit 9e80d7ed56439356d73846f479a91c7082e0b9c2
Author: Ken Huang <[email protected]>
AuthorDate: Tue Jan 21 01:22:40 2025 +0800
KAFKA-18363: Remove ZooKeeper mentiosn in broker configs (#18346)
Reviewers: Mickael Maison <[email protected]>, TengYao Chi
<[email protected]>
---
.../java/org/apache/kafka/server/config/QuotaConfig.java | 5 ++---
.../java/org/apache/kafka/network/SocketServerConfigs.java | 4 ++--
.../java/org/apache/kafka/server/config/KRaftConfigs.java | 14 ++++++--------
.../java/org/apache/kafka/server/config/ServerConfigs.java | 4 +---
4 files changed, 11 insertions(+), 16 deletions(-)
diff --git
a/server-common/src/main/java/org/apache/kafka/server/config/QuotaConfig.java
b/server-common/src/main/java/org/apache/kafka/server/config/QuotaConfig.java
index 6ecd7ffca3b..ddc9217057e 100644
---
a/server-common/src/main/java/org/apache/kafka/server/config/QuotaConfig.java
+++
b/server-common/src/main/java/org/apache/kafka/server/config/QuotaConfig.java
@@ -56,9 +56,8 @@ public class QuotaConfig {
public static final String CLIENT_QUOTA_CALLBACK_CLASS_CONFIG =
"client.quota.callback.class";
public static final String CLIENT_QUOTA_CALLBACK_CLASS_DOC = "The fully
qualified name of a class that implements the ClientQuotaCallback interface, " +
- "which is used to determine quota limits applied to client
requests. By default, the <user> and <client-id> " +
- "quotas that are stored in ZooKeeper are applied. For any given
request, the most specific quota that matches the user principal " +
- "of the session and the client-id of the request is applied.";
+ "which is used to determine quota limits applied to client
requests. " +
+ "For any given request, the most specific quota that matches the
user principal of the session and the client-id of the request is applied.";
public static final String LEADER_REPLICATION_THROTTLED_REPLICAS_CONFIG =
"leader.replication.throttled.replicas";
public static final String LEADER_REPLICATION_THROTTLED_REPLICAS_DOC = "A
list of replicas for which log replication should be throttled on " +
diff --git
a/server/src/main/java/org/apache/kafka/network/SocketServerConfigs.java
b/server/src/main/java/org/apache/kafka/network/SocketServerConfigs.java
index e7cb901bf19..e9c3dfff926 100644
--- a/server/src/main/java/org/apache/kafka/network/SocketServerConfigs.java
+++ b/server/src/main/java/org/apache/kafka/network/SocketServerConfigs.java
@@ -77,8 +77,8 @@ public class SocketServerConfigs {
public static final String ADVERTISED_LISTENERS_CONFIG =
"advertised.listeners";
public static final String ADVERTISED_LISTENERS_DOC =
String.format("Specifies the listener addresses that the Kafka brokers will
advertise to clients and other brokers." +
" The config is useful where the actual listener
configuration <code>%s</code> does not represent the addresses that clients
should" +
- " use to connect, such as in cloud environments. In
environments using ZooKeeper, these addresses are published to ZooKeeper." +
- " In Kraft mode, the address would be published to and
managed by kraft controller, the brokers would pull these data from controller
as needed." +
+ " use to connect, such as in cloud environments." +
+ " The addresses are published to and managed by the
controller, the brokers pull these data from the controller as needed." +
" In IaaS environments, this may need to be different from
the interface to which the broker binds." +
" If this is not set, the value for <code>%1$1s</code>
will be used." +
" Unlike <code>%1$1s</code>, it is not valid to advertise
the 0.0.0.0 meta-address.%n" +
diff --git
a/server/src/main/java/org/apache/kafka/server/config/KRaftConfigs.java
b/server/src/main/java/org/apache/kafka/server/config/KRaftConfigs.java
index a17cbb2131e..7a76a8d793c 100644
--- a/server/src/main/java/org/apache/kafka/server/config/KRaftConfigs.java
+++ b/server/src/main/java/org/apache/kafka/server/config/KRaftConfigs.java
@@ -35,20 +35,19 @@ import static
org.apache.kafka.common.config.ConfigDef.Type.STRING;
public class KRaftConfigs {
/** KRaft mode configs */
public static final String PROCESS_ROLES_CONFIG = "process.roles";
- public static final String PROCESS_ROLES_DOC = "The roles that this
process plays: 'broker', 'controller', or 'broker,controller' if it is both. " +
- "This configuration is only applicable for clusters in KRaft
(Kafka Raft) mode (instead of ZooKeeper). Leave this config undefined or empty
for ZooKeeper clusters.";
-
+ public static final String PROCESS_ROLES_DOC = "The roles that this
process plays: 'broker', 'controller', or 'broker,controller' if it is both. ";
+
public static final String INITIAL_BROKER_REGISTRATION_TIMEOUT_MS_CONFIG =
"initial.broker.registration.timeout.ms";
public static final int INITIAL_BROKER_REGISTRATION_TIMEOUT_MS_DEFAULT =
60000;
public static final String INITIAL_BROKER_REGISTRATION_TIMEOUT_MS_DOC =
"When initially registering with the controller quorum, the number of
milliseconds to wait before declaring failure and exiting the broker process.";
public static final String BROKER_HEARTBEAT_INTERVAL_MS_CONFIG =
"broker.heartbeat.interval.ms";
public static final int BROKER_HEARTBEAT_INTERVAL_MS_DEFAULT = 2000;
- public static final String BROKER_HEARTBEAT_INTERVAL_MS_DOC = "The length
of time in milliseconds between broker heartbeats. Used when running in KRaft
mode.";
+ public static final String BROKER_HEARTBEAT_INTERVAL_MS_DOC = "The length
of time in milliseconds between broker heartbeats.";
public static final String BROKER_SESSION_TIMEOUT_MS_CONFIG =
"broker.session.timeout.ms";
public static final int BROKER_SESSION_TIMEOUT_MS_DEFAULT = 9000;
- public static final String BROKER_SESSION_TIMEOUT_MS_DOC = "The length of
time in milliseconds that a broker lease lasts if no heartbeats are made. Used
when running in KRaft mode.";
+ public static final String BROKER_SESSION_TIMEOUT_MS_DOC = "The length of
time in milliseconds that a broker lease lasts if no heartbeats are made.";
public static final String NODE_ID_CONFIG = "node.id";
@@ -57,7 +56,7 @@ public class KRaftConfigs {
"This is required configuration when running in KRaft mode.";
public static final String METADATA_LOG_DIR_CONFIG = "metadata.log.dir";
- public static final String METADATA_LOG_DIR_DOC = "This configuration
determines where we put the metadata log for clusters in KRaft mode. " +
+ public static final String METADATA_LOG_DIR_DOC = "This configuration
determines where we put the metadata log. " +
"If it is not set, the metadata log is placed in the first log
directory from log.dirs.";
public static final String METADATA_SNAPSHOT_MAX_INTERVAL_MS_CONFIG =
"metadata.log.max.snapshot.interval.ms";
@@ -78,8 +77,7 @@ public class KRaftConfigs {
public static final String CONTROLLER_LISTENER_NAMES_CONFIG =
"controller.listener.names";
public static final String CONTROLLER_LISTENER_NAMES_DOC = "A
comma-separated list of the names of the listeners used by the controller. This
is required " +
- "if running in KRaft mode. When communicating with the controller
quorum, the broker will always use the first listener in this list.\n " +
- "Note: The ZooKeeper-based controller should not set this
configuration.";
+ "when communicating with the controller quorum, the broker will
always use the first listener in this list.";
public static final String SASL_MECHANISM_CONTROLLER_PROTOCOL_CONFIG =
"sasl.mechanism.controller.protocol";
public static final String SASL_MECHANISM_CONTROLLER_PROTOCOL_DOC = "SASL
mechanism used for communication with controllers. Default is GSSAPI.";
diff --git
a/server/src/main/java/org/apache/kafka/server/config/ServerConfigs.java
b/server/src/main/java/org/apache/kafka/server/config/ServerConfigs.java
index cd5da5ae9fd..e668db26e04 100644
--- a/server/src/main/java/org/apache/kafka/server/config/ServerConfigs.java
+++ b/server/src/main/java/org/apache/kafka/server/config/ServerConfigs.java
@@ -46,9 +46,7 @@ public class ServerConfigs {
public static final String BROKER_ID_CONFIG = "broker.id";
public static final int BROKER_ID_DEFAULT = -1;
- public static final String BROKER_ID_DOC = "The broker id for this server.
If unset, a unique broker id will be generated." +
- "To avoid conflicts between ZooKeeper generated broker id's and
user configured broker id's, generated broker ids " +
- "start from " + RESERVED_BROKER_MAX_ID_CONFIG + " + 1.";
+ public static final String BROKER_ID_DOC = "The broker id for this
server.";
public static final String MESSAGE_MAX_BYTES_CONFIG = "message.max.bytes";
public static final String MESSAGE_MAX_BYTES_DOC =
TopicConfig.MAX_MESSAGE_BYTES_DOC +