(pulsar) branch master updated: [fix] [broker] disable loadBalancerMemoryResourceWeight by default (#22820)

[xyz]

This is an automated email from the ASF dual-hosted git repository.

xyz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git


The following commit(s) were added to refs/heads/master by this push:
     new 342d88dd193 [fix] [broker] disable loadBalancerMemoryResourceWeight by 
default (#22820)
342d88dd193 is described below

commit 342d88dd193bb85c0af91c5193b1422808a9c821
Author: Wenzhi Feng <52550727+thetumb...@users.noreply.github.com>
AuthorDate: Wed Jun 5 09:51:33 2024 +0800

    [fix] [broker] disable loadBalancerMemoryResourceWeight by default (#22820)
---
 conf/broker.conf                                                        | 2 +-
 conf/standalone.conf                                                    | 2 +-
 deployment/terraform-ansible/templates/broker.conf                      | 2 +-
 .../src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java    | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/conf/broker.conf b/conf/broker.conf
index d68b6c6ca61..02e29402989 100644
--- a/conf/broker.conf
+++ b/conf/broker.conf
@@ -1816,7 +1816,7 @@ strictBookieAffinityEnabled=false
 # The heap memory usage weight when calculating new resource usage.
 # It only takes effect in the ThresholdShedder strategy.
 # Deprecated: Memory is no longer used as a load balancing item
-loadBalancerMemoryResourceWeight=1.0
+loadBalancerMemoryResourceWeight=0
 
 # Zookeeper quorum connection string
 # Deprecated: use metadataStoreUrl instead
diff --git a/conf/standalone.conf b/conf/standalone.conf
index 1a0c501899d..07d19c7bee9 100644
--- a/conf/standalone.conf
+++ b/conf/standalone.conf
@@ -924,7 +924,7 @@ loadBalancerCPUResourceWeight=1.0
 
 # The heap memory usage weight when calculating new resource usage.
 # It only takes effect in the ThresholdShedder strategy.
-loadBalancerMemoryResourceWeight=1.0
+loadBalancerMemoryResourceWeight=0
 
 # The direct memory usage weight when calculating new resource usage.
 # It only takes effect in the ThresholdShedder strategy.
diff --git a/deployment/terraform-ansible/templates/broker.conf 
b/deployment/terraform-ansible/templates/broker.conf
index 43bbdc0d52d..084d7f46ce1 100644
--- a/deployment/terraform-ansible/templates/broker.conf
+++ b/deployment/terraform-ansible/templates/broker.conf
@@ -967,7 +967,7 @@ loadBalancerCPUResourceWeight=1.0
 
 # The heap memory usage weight when calculating new resource usage.
 # It only take effect in ThresholdShedder strategy.
-loadBalancerMemoryResourceWeight=1.0
+loadBalancerMemoryResourceWeight=0
 
 # The direct memory usage weight when calculating new resource usage.
 # It only take effect in ThresholdShedder strategy.
diff --git 
a/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java
 
b/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java
index 6e8820db27c..204ea453bae 100644
--- 
a/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java
+++ 
b/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java
@@ -2469,7 +2469,7 @@ public class ServiceConfiguration implements 
PulsarConfiguration {
             doc = "Memory Resource Usage Weight. Deprecated: Memory is no 
longer used as a load balancing item.",
             deprecated = true
     )
-    private double loadBalancerMemoryResourceWeight = 1.0;
+    private double loadBalancerMemoryResourceWeight = 0;
 
     @FieldContext(
             dynamic = true,