JooHyukKim commented on code in PR #20552:
URL: https://github.com/apache/pulsar/pull/20552#discussion_r1225679695
##########
pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/extensions/ExtensibleLoadManagerImpl.java:
##########
@@ -180,6 +184,24 @@ public class ExtensibleLoadManagerImpl implements
ExtensibleLoadManager {
.build();
private final CountDownLatch initWaiter = new CountDownLatch(1);
+ /**
+ * Get all the bundles that are owned by this broker.
+ */
+ public Set<NamespaceBundle> getOwnedServiceUnits() {
+ var entrySet = serviceUnitStateChannel.getOwnershipEntrySet();
+ var brokerId = brokerRegistry.getBrokerId();
Review Comment:
> In fact, Pulsar already has been using "var".
Ah, is that so? Well, if that's the case, I will take back what I said 👍🏻
Thank you for letting me know.
But, may I ask in which class there is `var` usage? I also saw some in about
10 test classes, but not in production code yet. 🤔
--
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]
