added splittingBundles set to optimize

Author: heesung-sn

pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/extensions/strategy/DefaultNamespaceBundleSplitStrategyImpl.java

@@ -58,12 +58,14 @@ public class DefaultNamespaceBundleSplitStrategyImpl implements NamespaceBundleS
     private final Set<SplitDecision> decisionCache;
     private final Map<String, Integer> namespaceBundleCount;
     private final Map<String, Integer> splitConditionHitCounts;
+    private final Set<String> splittingBundles;
     private final SplitCounter counter;
 
     public DefaultNamespaceBundleSplitStrategyImpl(SplitCounter counter) {
         decisionCache = new HashSet<>();
         namespaceBundleCount = new HashMap<>();
         splitConditionHitCounts = new HashMap<>();
+        splittingBundles = new HashSet<>();
         this.counter = counter;
 
     }
@@ -72,6 +74,7 @@ public DefaultNamespaceBundleSplitStrategyImpl(SplitCounter counter) {
     public Set<SplitDecision> findBundlesToSplit(LoadManagerContext context, PulsarService pulsar) {
         decisionCache.clear();
         namespaceBundleCount.clear();
+        splittingBundles.clear();
         final ServiceConfiguration conf = pulsar.getConfiguration();
         int maxBundleCount = conf.getLoadBalancerNamespaceMaximumBundles();
         long maxBundleTopics = conf.getLoadBalancerNamespaceBundleMaxTopics();
@@ -83,6 +86,13 @@ public Set<SplitDecision> findBundlesToSplit(LoadManagerContext context, PulsarS
         boolean debug = log.isDebugEnabled() || conf.isLoadBalancerDebugModeEnabled();
         var channel = ServiceUnitStateChannelImpl.get(pulsar);
 
+        for (var etr : channel.getOwnershipEntrySet()) {
+            var eData = etr.getValue();
+            if (eData.state() == ServiceUnitState.Splitting) {
+                splittingBundles.add(etr.getKey());
+            }
+        }
+
         Map<String, NamespaceBundleStats> bundleStatsMap = pulsar.getBrokerService().getBundleStats();
         NamespaceBundleFactory namespaceBundleFactory =
                 pulsar.getNamespaceService().getNamespaceBundleFactory();
@@ -185,15 +195,14 @@ public Set<SplitDecision> findBundlesToSplit(LoadManagerContext context, PulsarS
 
             var ranges = bundleRange.split("_");
             var foundSplittingBundle = false;
-            for (var etr : channel.getOwnershipEntrySet()) {
-                var eBundle = etr.getKey();
-                var eData = etr.getValue();
-                if (eData.state() == ServiceUnitState.Splitting && eBundle.startsWith(namespace)) {
-                    final String eRange = LoadManagerShared.getBundleRangeFromBundleName(eBundle);
-                    if (eRange.startsWith(ranges[0]) || eRange.endsWith(ranges[1])) {
+            for (var splittingBundle : splittingBundles) {
+                if (splittingBundle.startsWith(namespace)) {
+                    final String splittingBundleRange = LoadManagerShared.getBundleRangeFromBundleName(splittingBundle);
+                    if (splittingBundleRange.startsWith(ranges[0])
+                            || splittingBundleRange.endsWith(ranges[1])) {
                         if (debug) {
                             log.info(String.format(CANNOT_SPLIT_BUNDLE_MSG
-                                    + " (parent) bundle:%s is in Splitting state.", bundle, eBundle));
+                                    + " (parent) bundle:%s is in Splitting state.", bundle, splittingBundle));
                         }
                         foundSplittingBundle = true;
                         break;