gaozhangmin commented on a change in pull request #12957:
URL: https://github.com/apache/pulsar/pull/12957#discussion_r757416667
##########
File path:
pulsar-broker/src/test/java/org/apache/pulsar/broker/namespace/NamespaceServiceTest.java
##########
@@ -570,13 +570,15 @@ public void testSplitBundleWithHighestThroughput() throws
Exception {
NamespaceBundles bundles =
namespaceService.getNamespaceBundleFactory().getBundles(nsname);
String bundle = bundles.findBundle(TopicName.get(topic +
"0")).getBundleRange();
- String path = ModularLoadManagerImpl.getBundleDataPath(bundle);
+ String path = ModularLoadManagerImpl.getBundleDataPath(namespace + "/"
+ bundle);
NamespaceBundleStats defaultStats = new NamespaceBundleStats();
defaultStats.msgThroughputIn = 100000;
defaultStats.msgThroughputOut = 100000;
- BundleData bd = new BundleData(10, 19, defaultStats );
+ BundleData bd = new BundleData(10, 19, defaultStats);
+ bd.setTopics(10);
byte[] data =
ObjectMapperFactory.getThreadLocal().writeValueAsBytes(bd);
pulsar.getLocalMetadataStore().put(path, data, Optional.empty());
+ Thread.sleep(3);
Review comment:
1、 Incorrect bundle path, when getting bundle-data from zk. missed
namespace info.
2、Split an unowned bundle is not allowed. which
getBundleWithHighestThroughput would return.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
