From d0bca67b27476a30c462a6bb3c172daa37367a4e Mon Sep 17 00:00:00 2001 From: Ken Huang Date: Tue, 21 Jan 2025 01:22:40 +0800 Subject: [PATCH] KAFKA-18363: Remove ZooKeeper mentiosn in broker configs (#18346) Reviewers: Mickael Maison , TengYao Chi --- .../apache/kafka/server/config/QuotaConfig.java | 5 ++--- .../apache/kafka/network/SocketServerConfigs.java | 4 ++-- .../apache/kafka/server/config/KRaftConfigs.java | 14 ++++++-------- .../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 4480e9f0c10..cc2c3730813 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 @@ -54,9 +54,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 %s 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 %1$1s will be used." + " Unlike %1$1s, 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 3f07c0b7547..bb7d9234ea4 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 +