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

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


The following commit(s) were added to refs/heads/master by this push:
     new 62ea072854 fix a config error (#4098)
62ea072854 is described below

commit 62ea072854d8de5b0dc363d3e63909bf6a7f2aea
Author: ken <[email protected]>
AuthorDate: Mon Jan 8 12:18:14 2024 +0800

    fix a config error (#4098)
    
    Co-authored-by: fanjianye <[email protected]>
    ### Motivation
    
    fix a config error in RegionAware#initialize.
    
    There is double config ignoreLocalNodeInPlacementPolicy in 
RegionAware#initialize. Actually one of
     the config is useHostnameResolveLocalNodePlacementPolicy.
    
    ### change
    
    use correct config name
---
 .../apache/bookkeeper/client/RegionAwareEnsemblePlacementPolicy.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/RegionAwareEnsemblePlacementPolicy.java
 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/RegionAwareEnsemblePlacementPolicy.java
index f9d80d1469..c742e62c04 100644
--- 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/RegionAwareEnsemblePlacementPolicy.java
+++ 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/client/RegionAwareEnsemblePlacementPolicy.java
@@ -244,7 +244,7 @@ public class RegionAwareEnsemblePlacementPolicy extends 
RackawareEnsemblePlaceme
                         .initialize(dnsResolver, timer, 
this.reorderReadsRandom, this.stabilizePeriodSeconds,
                                 this.reorderThresholdPendingRequests, 
this.isWeighted, this.maxWeightMultiple,
                                 this.minNumRacksPerWriteQuorum, 
this.enforceMinNumRacksPerWriteQuorum,
-                                this.ignoreLocalNodeInPlacementPolicy, 
this.ignoreLocalNodeInPlacementPolicy,
+                                this.ignoreLocalNodeInPlacementPolicy, 
this.useHostnameResolveLocalNodePlacementPolicy,
                                 statsLogger, bookieAddressResolver)
                         
.withDefaultRack(NetworkTopology.DEFAULT_REGION_AND_RACK));
             }

Reply via email to