You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by kl...@apache.org on 2016/02/06 02:25:05 UTC

incubator-geode git commit: Fix crashDistributedSystem use of invoke

Repository: incubator-geode
Updated Branches:
  refs/heads/feature/GEODE-773-2 30f29f44f -> 91ab06d3c


Fix crashDistributedSystem use of invoke


Project: http://git-wip-us.apache.org/repos/asf/incubator-geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/91ab06d3
Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/91ab06d3
Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/91ab06d3

Branch: refs/heads/feature/GEODE-773-2
Commit: 91ab06d3c4ec6aace9dcccc76ca936ad791ed0be
Parents: 30f29f4
Author: Kirk Lund <kl...@pivotal.io>
Authored: Fri Feb 5 17:24:32 2016 -0800
Committer: Kirk Lund <kl...@pivotal.io>
Committed: Fri Feb 5 17:24:32 2016 -0800

----------------------------------------------------------------------
 .../gemstone/gemfire/test/dunit/DistributedTestUtils.java    | 8 +++-----
 gemfire-core/src/test/java/hydra/MethExecutor.java           | 1 +
 2 files changed, 4 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/91ab06d3/gemfire-core/src/test/java/com/gemstone/gemfire/test/dunit/DistributedTestUtils.java
----------------------------------------------------------------------
diff --git a/gemfire-core/src/test/java/com/gemstone/gemfire/test/dunit/DistributedTestUtils.java b/gemfire-core/src/test/java/com/gemstone/gemfire/test/dunit/DistributedTestUtils.java
index 39c8ab3..a040a32 100755
--- a/gemfire-core/src/test/java/com/gemstone/gemfire/test/dunit/DistributedTestUtils.java
+++ b/gemfire-core/src/test/java/com/gemstone/gemfire/test/dunit/DistributedTestUtils.java
@@ -98,17 +98,15 @@ public class DistributedTestUtils {
    * NOTE: if you use this method be sure that you clean up the VM before the end of your
    * test with disconnectFromDS() or disconnectAllFromDS().
    */
-  @SuppressWarnings("serial")
   public static boolean crashDistributedSystem(final VM vm) {
-    return vm.invoke(new SerializableCallable<Boolean>(DistributedTestUtils.class.getSimpleName() + " crashDistributedSystem") {
-      public Boolean call() throws Exception {
+    return vm.invoke(()->{
         DistributedSystem system = InternalDistributedSystem.getAnyInstance();
         crashDistributedSystem(system);
         return true;
       }
-    });
+    );
   }
-
+  
   /** 
    * Delete locator state files.  Use this after getting a random port
    * to ensure that an old locator state file isn't picked up by the

http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/91ab06d3/gemfire-core/src/test/java/hydra/MethExecutor.java
----------------------------------------------------------------------
diff --git a/gemfire-core/src/test/java/hydra/MethExecutor.java b/gemfire-core/src/test/java/hydra/MethExecutor.java
index c38a803..8aaf3dc 100644
--- a/gemfire-core/src/test/java/hydra/MethExecutor.java
+++ b/gemfire-core/src/test/java/hydra/MethExecutor.java
@@ -147,6 +147,7 @@ public class MethExecutor {
       }
       sb.append("] in class ");
       sb.append(c.getName());
+      sb.append(" methods=" + matchingMethods);
       throw new NoSuchMethodException(sb.toString());
     }
     else return (Method) matchingMethods.get(0);