You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by wi...@apache.org on 2018/12/14 23:38:03 UTC

[geode] branch develop updated: GEODE-5674: Stop picking ports randomly for tests (#3007)

This is an automated email from the ASF dual-hosted git repository.

wirebaron pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/develop by this push:
     new e7181ee  GEODE-5674: Stop picking ports randomly for tests (#3007)
e7181ee is described below

commit e7181ee37cbf1edebd631f858eff728f97d4cfd7
Author: Brian Rowe <br...@pivotal.io>
AuthorDate: Fri Dec 14 15:37:54 2018 -0800

    GEODE-5674: Stop picking ports randomly for tests (#3007)
    
    Removing an accidental ++ that was breaking the previous commit.
---
 .../src/main/java/org/apache/geode/internal/AvailablePortHelper.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/geode-junit/src/main/java/org/apache/geode/internal/AvailablePortHelper.java b/geode-junit/src/main/java/org/apache/geode/internal/AvailablePortHelper.java
index 10047c2..dd75c47 100644
--- a/geode-junit/src/main/java/org/apache/geode/internal/AvailablePortHelper.java
+++ b/geode-junit/src/main/java/org/apache/geode/internal/AvailablePortHelper.java
@@ -282,7 +282,7 @@ public class AvailablePortHelper {
         continue;
       }
 
-      if (AvailablePort.isPortAvailable(uniquePort++, protocol, getAddress(protocol))) {
+      if (AvailablePort.isPortAvailable(uniquePort, protocol, getAddress(protocol))) {
         return uniquePort;
       }
     }