You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by kh...@apache.org on 2017/11/20 21:24:22 UTC

[geode] branch develop updated: GEODE-3980: Remove unneeded additional findAvailablePids calls (#1076)

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

khowe 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 0dd3c91  GEODE-3980: Remove unneeded additional findAvailablePids calls (#1076)
0dd3c91 is described below

commit 0dd3c91a823c1f92debfcb3c25cb176e34a632b0
Author: Kenneth Howe <kh...@pivotal.io>
AuthorDate: Mon Nov 20 13:24:20 2017 -0800

    GEODE-3980: Remove unneeded additional findAvailablePids calls (#1076)
    
    The extra findAvailablePids calls result in flaky results on
    Windows.
---
 .../java/org/apache/geode/internal/process/lang/AvailablePidTest.java   | 2 --
 1 file changed, 2 deletions(-)

diff --git a/geode-core/src/test/java/org/apache/geode/internal/process/lang/AvailablePidTest.java b/geode-core/src/test/java/org/apache/geode/internal/process/lang/AvailablePidTest.java
index 48ba235..62a5fd3 100644
--- a/geode-core/src/test/java/org/apache/geode/internal/process/lang/AvailablePidTest.java
+++ b/geode-core/src/test/java/org/apache/geode/internal/process/lang/AvailablePidTest.java
@@ -97,7 +97,6 @@ public class AvailablePidTest {
     assertThat(availablePid.findAvailablePids(2)).hasSize(2);
     assertThat(availablePid.findAvailablePids(3)).hasSize(3);
     assertThat(availablePid.findAvailablePids(5)).hasSize(5);
-    assertThat(availablePid.findAvailablePids(8)).hasSize(8);
   }
 
   @Test
@@ -106,7 +105,6 @@ public class AvailablePidTest {
     assertThatNoPidIsDuplicated(availablePid.findAvailablePids(2));
     assertThatNoPidIsDuplicated(availablePid.findAvailablePids(3));
     assertThatNoPidIsDuplicated(availablePid.findAvailablePids(5));
-    assertThatNoPidIsDuplicated(availablePid.findAvailablePids(8));
   }
 
   @Test

-- 
To stop receiving notification emails like this one, please contact
['"commits@geode.apache.org" <co...@geode.apache.org>'].