From 7f2a88df1b4af8718f1d0ae1e085db8aaa62650a Mon Sep 17 00:00:00 2001 From: Zachary Pinto Date: Thu, 2 Nov 2023 11:47:58 -0700 Subject: [PATCH] Fix condition to check if DOMAIN has all required TOPOLOGY keys and allows for extra kv pairs, to reduce noisy logs. (#2688) --- .../apache/helix/controller/rebalancer/topology/Topology.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/helix-core/src/main/java/org/apache/helix/controller/rebalancer/topology/Topology.java b/helix-core/src/main/java/org/apache/helix/controller/rebalancer/topology/Topology.java index dba1201680..4d4ebabd11 100644 --- a/helix-core/src/main/java/org/apache/helix/controller/rebalancer/topology/Topology.java +++ b/helix-core/src/main/java/org/apache/helix/controller/rebalancer/topology/Topology.java @@ -233,7 +233,7 @@ private static LinkedHashMap computeInstanceTopologyMapHelper( return instanceTopologyMap; } } - if (numOfMatchedKeys != domainAsMap.size()) { + if (numOfMatchedKeys < clusterTopologyConfig.getTopologyKeyDefaultValue().size()) { logger.warn( "Key-value pairs in InstanceConfig.Domain {} do not align with keys in ClusterConfig.Topology " + "{}, using default domain value instead", instanceConfig.getDomainAsString(),