upthewaterspout commented on pull request #5598: URL: https://github.com/apache/geode/pull/5598#issuecomment-704445463
BTW, the actual product change is just shifting some parenthesis around in an if statement. Most of these changes have to do with moving AvailablePort so that I could write a good test. We need to be able to write a test at the membership level where two locators are started up that both know about each other (including their ports) at startup. The best way to do that is to use AvailablePort to get the ports before starting the locators. ---------------------------------------------------------------- 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]
