Skip to content

Commit

Permalink
Fix condition to check if DOMAIN has all required TOPOLOGY keys and a…
Browse files Browse the repository at this point in the history
…llows for extra kv pairs, to reduce noisy logs. (#2688)
  • Loading branch information
zpinto authored Nov 2, 2023
1 parent bd1d28c commit 7f2a88d
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -233,7 +233,7 @@ private static LinkedHashMap<String, String> 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(),
Expand Down

0 comments on commit 7f2a88d

Please sign in to comment.