Bill commented on a change in pull request #6472:
URL: https://github.com/apache/geode/pull/6472#discussion_r632142518
##########
File path:
geode-tcp-server/src/distributedTest/java/org/apache/geode/distributed/internal/tcpserver/TcpServerProductVersionDUnitTest.java
##########
@@ -251,17 +251,15 @@ private TcpClient getTcpClient() {
TcpSocketFactory.DEFAULT);
}
- private int createLocator(VM memberVM, boolean usingOldVersion) {
+ private int createLocator(VM memberVM) {
+ final int port = AvailablePortHelper.getRandomAvailableTCPPort();
Review comment:
One minor tweak might be to move this definition closer to the first
point of use down on line 262.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
