From d963939bb9284a94b87d0560ed537b5466557cc6 Mon Sep 17 00:00:00 2001 From: SEPURI-SAI-KRISHNA Date: Sat, 8 Aug 2026 12:06:03 +0530 Subject: [PATCH] [cleanup][broker] Remove configuration entries that no longer exist in the code Assisted-by: Claude Code --- conf/broker.conf | 7 ------- conf/standalone.conf | 7 ------- deployment/terraform-ansible/templates/broker.conf | 4 ---- .../tests/integration/plugins/TestBrokerInterceptors.java | 1 - 4 files changed, 19 deletions(-) diff --git a/conf/broker.conf b/conf/broker.conf index d1ab312300c18..2ac727e08d105 100644 --- a/conf/broker.conf +++ b/conf/broker.conf @@ -1244,10 +1244,6 @@ bookkeeperClientIsolationGroups= # have enough bookie available. bookkeeperClientSecondaryIsolationGroups= -# Minimum bookies that should be available as part of bookkeeperClientIsolationGroups -# else broker will include bookkeeperClientSecondaryIsolationGroups bookies in isolated list. -bookkeeperClientMinAvailableBookiesInIsolationGroups= - # Enable/disable having read operations for a ledger to be sticky to a single bookie. # If this flag is enabled, the client will use one single bookie (by preference) to read # all entries for a ledger. @@ -2263,6 +2259,3 @@ brokerInterceptorsDirectory=./interceptors # List of broker interceptor to load, which is a list of broker interceptor names brokerInterceptors= - -# Enable or disable the broker interceptor, which is only used for testing for now -disableBrokerInterceptors=true diff --git a/conf/standalone.conf b/conf/standalone.conf index eed7af99e5ada..fe4ec81279a24 100644 --- a/conf/standalone.conf +++ b/conf/standalone.conf @@ -798,10 +798,6 @@ bookkeeperClientIsolationGroups= # have enough bookie available. bookkeeperClientSecondaryIsolationGroups= -# Minimum bookies that should be available as part of bookkeeperClientIsolationGroups -# else broker will include bookkeeperClientSecondaryIsolationGroups bookies in isolated list. -bookkeeperClientMinAvailableBookiesInIsolationGroups= - # Set the client security provider factory class name. # Default: org.apache.bookkeeper.tls.TLSContextFactory bookkeeperTLSProviderFactoryClass=org.apache.bookkeeper.tls.TLSContextFactory @@ -1555,9 +1551,6 @@ brokerInterceptorsDirectory=./interceptors # List of broker interceptor to load, which is a list of broker interceptor names brokerInterceptors= -# Enable or disable the broker interceptor, which is only used for testing for now -disableBrokerInterceptors=true - # Whether retain null-key message during topic compaction topicCompactionRetainNullKey=false diff --git a/deployment/terraform-ansible/templates/broker.conf b/deployment/terraform-ansible/templates/broker.conf index 51d5bbae0316c..ad4682e309511 100644 --- a/deployment/terraform-ansible/templates/broker.conf +++ b/deployment/terraform-ansible/templates/broker.conf @@ -693,10 +693,6 @@ bookkeeperClientIsolationGroups= # have enough bookie available. bookkeeperClientSecondaryIsolationGroups= -# Minimum bookies that should be available as part of bookkeeperClientIsolationGroups -# else broker will include bookkeeperClientSecondaryIsolationGroups bookies in isolated list. -bookkeeperClientMinAvailableBookiesInIsolationGroups= - # Enable/disable having read operations for a ledger to be sticky to a single bookie. # If this flag is enabled, the client will use one single bookie (by preference) to read # all entries for a ledger. diff --git a/tests/integration/src/test/java/org/apache/pulsar/tests/integration/plugins/TestBrokerInterceptors.java b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/plugins/TestBrokerInterceptors.java index 513d77307e642..bd1cf4b279720 100644 --- a/tests/integration/src/test/java/org/apache/pulsar/tests/integration/plugins/TestBrokerInterceptors.java +++ b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/plugins/TestBrokerInterceptors.java @@ -41,7 +41,6 @@ public class TestBrokerInterceptors extends TopicMessagingBase { @Override public void setupCluster() throws Exception { - brokerEnvs.put(PREFIX + "disableBrokerInterceptors", "false"); brokerEnvs.put(PREFIX + "brokerInterceptorsDirectory", "/pulsar/examples"); brokerEnvs.put(PREFIX + "brokerInterceptors", "loggingInterceptor"); super.setupCluster();