berg223 commented on code in PR #24371:
URL: https://github.com/apache/pulsar/pull/24371#discussion_r2134455733
##########
pulsar-broker/src/test/java/org/apache/pulsar/broker/loadbalance/extensions/ExtensibleLoadManagerImplTest.java:
##########
@@ -837,7 +845,8 @@ public static void
testOptimizeUnloadDisable(List<PulsarClient> clients,
}
}
- protected static Pair<LookupService, LookupService>
spyLookupService(PulsarClient client) throws IllegalAccessException {
+ protected static Pair<LookupService, LookupService>
spyLookupService(PulsarClient client)
+ throws IllegalAccessException {
Review Comment:
> It looks like there are unnecessary formatting changes. In Pulsar, we use
the line width of 120. Please ensure that you have the code style setup in the
way described in https://pulsar.apache.org/contribute/setup-ide/. Most
contributors use IntelliJ.
I have the code style setup before fomatting according to the doc you point
out. Actually, the line has 121 characters. So it should be formatted! Let's
skip the formmatting changes!
--
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]
