Address comments

Author: Demogorgon314

pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/extensions/channel/ServiceUnitStateData.java

@@ -46,15 +46,15 @@ public ServiceUnitStateData(ServiceUnitState state, String dstBroker, String sou
     }
 
     public ServiceUnitStateData(ServiceUnitState state, String dstBroker, String sourceBroker, long versionId) {
-        this(state, dstBroker, sourceBroker, Map.of(), false, System.currentTimeMillis(), versionId);
+        this(state, dstBroker, sourceBroker, null, false, System.currentTimeMillis(), versionId);
     }
 
     public ServiceUnitStateData(ServiceUnitState state, String dstBroker, long versionId) {
-        this(state, dstBroker, null, Map.of(), false, System.currentTimeMillis(), versionId);
+        this(state, dstBroker, null, null, false, System.currentTimeMillis(), versionId);
     }
 
     public ServiceUnitStateData(ServiceUnitState state, String dstBroker, boolean force, long versionId) {
-        this(state, dstBroker, null, Map.of(), force, System.currentTimeMillis(), versionId);
+        this(state, dstBroker, null, null, force, System.currentTimeMillis(), versionId);
     }
 
     public static ServiceUnitState state(ServiceUnitStateData data) {

pulsar-broker/src/test/java/org/apache/pulsar/broker/loadbalance/extensions/ExtensibleLoadManagerImplTest.java

@@ -385,12 +385,22 @@ public void testSplitBundleAdminAPI() throws Exception {
         admin.topics().createPartitionedTopic(topic, 10);
         BundlesData bundles = admin.namespaces().getBundles(namespace);
         int numBundles = bundles.getNumBundles();
+        var bundleRanges = bundles.getBoundaries().stream().map(Long::decode).sorted().toList();
+
+        String firstBundle = bundleRanges.get(0) + "_" + bundleRanges.get(1);
 
-        String firstBundle = bundles.getBoundaries().get(0) + "_" + bundles.getBoundaries().get(1);
+        long mid = bundleRanges.get(0) + (bundleRanges.get(1) - bundleRanges.get(0)) / 2;
 
         admin.namespaces().splitNamespaceBundle(namespace, firstBundle, true, null);
 
-        assertEquals(admin.namespaces().getBundles(namespace).getNumBundles(), numBundles + 1);
+        BundlesData bundlesData = admin.namespaces().getBundles(namespace);
+        assertEquals(bundlesData.getNumBundles(), numBundles + 1);
+        String lowBundle = String.format("0x%08x", bundleRanges.get(0));
+        String midBundle = String.format("0x%08x", mid);
+        String highBundle = String.format("0x%08x", bundleRanges.get(1));
+        assertTrue(bundlesData.getBoundaries().contains(lowBundle));
+        assertTrue(bundlesData.getBoundaries().contains(midBundle));
+        assertTrue(bundlesData.getBoundaries().contains(highBundle));
 
         // Test split bundle with invalid bundle range.
         try {