AuroraTwinkle commented on code in PR #24574:
URL: https://github.com/apache/pulsar/pull/24574#discussion_r2239441925
##########
pulsar-broker/src/test/java/org/apache/pulsar/client/impl/ServiceUrlQuarantineTest.java:
##########
@@ -53,10 +53,11 @@ public class ServiceUrlQuarantineTest extends
ProducerConsumerBase {
private PulsarClientImpl pulsarClientWithBinaryServiceUrlDisableQuarantine;
private PulsarClientImpl pulsarClientWithHttpServiceUrl;
private PulsarClientImpl pulsarClientWithHttpServiceUrlDisableQuarantine;
- private static final int BROKER_SERVICE_PORT = 6666;
- private static final int WEB_SERVICE_PORT = 8888;
+ private final int brokerServicePort = PortManager.nextLockedFreePort();
+ private final int webServicePort = PortManager.nextLockedFreePort();
Review Comment:
> Set these in the setup method. We have retries for TestNG to address
flakiness so that's why it's better to handle it that way.
OK, fixed!
--
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]
