You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ignite.apache.org by sb...@apache.org on 2015/07/10 17:25:51 UTC
incubator-ignite git commit: # ignite-648: review
Repository: incubator-ignite
Updated Branches:
refs/heads/ignite-648 9c96a2f06 -> 1e7889b5e
# ignite-648: review
Project: http://git-wip-us.apache.org/repos/asf/incubator-ignite/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-ignite/commit/1e7889b5
Tree: http://git-wip-us.apache.org/repos/asf/incubator-ignite/tree/1e7889b5
Diff: http://git-wip-us.apache.org/repos/asf/incubator-ignite/diff/1e7889b5
Branch: refs/heads/ignite-648
Commit: 1e7889b5e1c5b47a3132cf829dc159b913b726ec
Parents: 9c96a2f
Author: ashutak <as...@gridgain.com>
Authored: Fri Jul 10 18:25:52 2015 +0300
Committer: ashutak <as...@gridgain.com>
Committed: Fri Jul 10 18:25:52 2015 +0300
----------------------------------------------------------------------
.../ignite/testframework/junits/GridAbstractTest.java | 2 +-
.../junits/multijvm/IgniteProcessProxy.java | 13 ++++++++++++-
2 files changed, 13 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/1e7889b5/modules/core/src/test/java/org/apache/ignite/testframework/junits/GridAbstractTest.java
----------------------------------------------------------------------
diff --git a/modules/core/src/test/java/org/apache/ignite/testframework/junits/GridAbstractTest.java b/modules/core/src/test/java/org/apache/ignite/testframework/junits/GridAbstractTest.java
index 3d27d2d..06a1523 100644
--- a/modules/core/src/test/java/org/apache/ignite/testframework/junits/GridAbstractTest.java
+++ b/modules/core/src/test/java/org/apache/ignite/testframework/junits/GridAbstractTest.java
@@ -1383,7 +1383,7 @@ public abstract class GridAbstractTest extends TestCase {
* @return <code>True</code> if test was run in multi-JVM mode and grid with this name was started at another JVM.
*/
protected boolean isRemoteJvm(String gridName) {
- return isMultiJvm() && !gridName.endsWith("0");
+ return isMultiJvm() && !"0".equals(gridName.substring(getTestGridName().length()));
}
/**
http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/1e7889b5/modules/core/src/test/java/org/apache/ignite/testframework/junits/multijvm/IgniteProcessProxy.java
----------------------------------------------------------------------
diff --git a/modules/core/src/test/java/org/apache/ignite/testframework/junits/multijvm/IgniteProcessProxy.java b/modules/core/src/test/java/org/apache/ignite/testframework/junits/multijvm/IgniteProcessProxy.java
index ff1b62b..220424a 100644
--- a/modules/core/src/test/java/org/apache/ignite/testframework/junits/multijvm/IgniteProcessProxy.java
+++ b/modules/core/src/test/java/org/apache/ignite/testframework/junits/multijvm/IgniteProcessProxy.java
@@ -103,7 +103,18 @@ public class IgniteProcessProxy implements IgniteEx {
assert rmtNodeStartedLatch.await(30, TimeUnit.SECONDS): "Remote node has not joined [id=" + id + ']';
- gridProxies.put(cfg.getGridName(), this);
+ IgniteProcessProxy prevVal = gridProxies.putIfAbsent(cfg.getGridName(), this);
+
+ if (prevVal != null) {
+ remoteCompute().run(new IgniteRunnable() {
+ @Override public void run() {
+ G.stop(cfg.getGridName(), true);
+ }
+ });
+
+ throw new IllegalStateException("There was found instance assotiated with " + cfg.getGridName() +
+ ", instance= " + prevVal + ". New started node was stopped.");
+ }
}
/**