# IGNITE-405: Disabled external executino and removed it from public API.

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

Branch: refs/heads/ignite-329
Commit: dab66dc1b8b755f03ac3675b9193424791d7324b
Parents: 40b96e1
Author: vozerov-gridgain <voze...@gridgain.com>
Authored: Tue Mar 10 13:27:33 2015 +0300
Committer: vozerov-gridgain <voze...@gridgain.com>
Committed: Tue Mar 10 13:29:18 2015 +0300

----------------------------------------------------------------------
 .../ignite/configuration/HadoopConfiguration.java  | 17 ++++++++++-------
 .../processors/hadoop/HadoopProcessor.java         |  4 +++-
 .../HadoopClientProtocolEmbeddedSelfTest.java      |  3 ++-
 .../processors/hadoop/HadoopGroupingTest.java      |  3 ++-
 .../hadoop/HadoopJobTrackerSelfTest.java           |  4 +++-
 .../hadoop/HadoopMapReduceEmbeddedSelfTest.java    |  3 ++-
 .../hadoop/HadoopSortingExternalTest.java          |  3 ++-
 .../processors/hadoop/HadoopSortingTest.java       |  3 ++-
 .../hadoop/HadoopTaskExecutionSelfTest.java        |  4 +++-
 .../HadoopExternalTaskExecutionSelfTest.java       |  3 ++-
 .../ignite/testsuites/IgniteHadoopTestSuite.java   |  6 +++---
 11 files changed, 34 insertions(+), 19 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/dab66dc1/modules/core/src/main/java/org/apache/ignite/configuration/HadoopConfiguration.java
----------------------------------------------------------------------
diff --git 
a/modules/core/src/main/java/org/apache/ignite/configuration/HadoopConfiguration.java
 
b/modules/core/src/main/java/org/apache/ignite/configuration/HadoopConfiguration.java
index 01ef8b0..3a7f877 100644
--- 
a/modules/core/src/main/java/org/apache/ignite/configuration/HadoopConfiguration.java
+++ 
b/modules/core/src/main/java/org/apache/ignite/configuration/HadoopConfiguration.java
@@ -65,7 +65,8 @@ public class HadoopConfiguration {
      */
     public HadoopConfiguration(HadoopConfiguration cfg) {
         // Preserve alphabetic order.
-        extExecution = cfg.isExternalExecution();
+        // TODO: IGNITE-404: Uncomment when fixed.
+        //extExecution = cfg.isExternalExecution();
         finishedJobInfoTtl = cfg.getFinishedJobInfoTtl();
         planner = cfg.getMapReducePlanner();
         maxParallelTasks = cfg.getMaxParallelTasks();
@@ -132,9 +133,10 @@ public class HadoopConfiguration {
      *
      * @return {@code True} if external execution.
      */
-    public boolean isExternalExecution() {
-        return extExecution;
-    }
+    // TODO: IGNITE-404: Uncomment when fixed.
+//    public boolean isExternalExecution() {
+//        return extExecution;
+//    }
 
     /**
      * Sets external task execution flag.
@@ -142,9 +144,10 @@ public class HadoopConfiguration {
      * @param extExecution {@code True} if tasks should be executed in an 
external process.
      * @see #isExternalExecution()
      */
-    public void setExternalExecution(boolean extExecution) {
-        this.extExecution = extExecution;
-    }
+    // TODO: IGNITE-404: Uncomment when fixed.
+//    public void setExternalExecution(boolean extExecution) {
+//        this.extExecution = extExecution;
+//    }
 
     /**
      * Gets Hadoop map-reduce planner, a component which defines job execution 
plan based on job

http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/dab66dc1/modules/hadoop/src/main/java/org/apache/ignite/internal/processors/hadoop/HadoopProcessor.java
----------------------------------------------------------------------
diff --git 
a/modules/hadoop/src/main/java/org/apache/ignite/internal/processors/hadoop/HadoopProcessor.java
 
b/modules/hadoop/src/main/java/org/apache/ignite/internal/processors/hadoop/HadoopProcessor.java
index e0c5916..9bb6a0e 100644
--- 
a/modules/hadoop/src/main/java/org/apache/ignite/internal/processors/hadoop/HadoopProcessor.java
+++ 
b/modules/hadoop/src/main/java/org/apache/ignite/internal/processors/hadoop/HadoopProcessor.java
@@ -91,7 +91,9 @@ public class HadoopProcessor extends HadoopProcessorAdapter {
                 ctx,
                 cfg,
                 new HadoopJobTracker(),
-                cfg.isExternalExecution() ? new HadoopExternalTaskExecutor() : 
new HadoopEmbeddedTaskExecutor(),
+                new HadoopEmbeddedTaskExecutor(),
+                // TODO: IGNITE-404: Uncomment when fixed.
+                //cfg.isExternalExecution() ? new HadoopExternalTaskExecutor() 
: new HadoopEmbeddedTaskExecutor(),
                 new HadoopShuffle());
 
 

http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/dab66dc1/modules/hadoop/src/test/java/org/apache/ignite/client/hadoop/HadoopClientProtocolEmbeddedSelfTest.java
----------------------------------------------------------------------
diff --git 
a/modules/hadoop/src/test/java/org/apache/ignite/client/hadoop/HadoopClientProtocolEmbeddedSelfTest.java
 
b/modules/hadoop/src/test/java/org/apache/ignite/client/hadoop/HadoopClientProtocolEmbeddedSelfTest.java
index ffa20d1..a546559 100644
--- 
a/modules/hadoop/src/test/java/org/apache/ignite/client/hadoop/HadoopClientProtocolEmbeddedSelfTest.java
+++ 
b/modules/hadoop/src/test/java/org/apache/ignite/client/hadoop/HadoopClientProtocolEmbeddedSelfTest.java
@@ -27,7 +27,8 @@ public class HadoopClientProtocolEmbeddedSelfTest extends 
HadoopClientProtocolSe
     @Override public HadoopConfiguration hadoopConfiguration(String gridName) {
         HadoopConfiguration cfg = super.hadoopConfiguration(gridName);
 
-        cfg.setExternalExecution(false);
+        // TODO: IGNITE-404: Uncomment when fixed.
+        //cfg.setExternalExecution(false);
 
         return cfg;
     }

http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/dab66dc1/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopGroupingTest.java
----------------------------------------------------------------------
diff --git 
a/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopGroupingTest.java
 
b/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopGroupingTest.java
index e385ca7..f2abf12 100644
--- 
a/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopGroupingTest.java
+++ 
b/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopGroupingTest.java
@@ -64,7 +64,8 @@ public class HadoopGroupingTest extends 
HadoopAbstractSelfTest {
     @Override public HadoopConfiguration hadoopConfiguration(String gridName) {
         HadoopConfiguration cfg = super.hadoopConfiguration(gridName);
 
-        cfg.setExternalExecution(false);
+        // TODO: IGNITE-404: Uncomment when fixed.
+        //cfg.setExternalExecution(false);
 
         return cfg;
     }

http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/dab66dc1/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopJobTrackerSelfTest.java
----------------------------------------------------------------------
diff --git 
a/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopJobTrackerSelfTest.java
 
b/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopJobTrackerSelfTest.java
index 943d89f..6cf0e5e 100644
--- 
a/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopJobTrackerSelfTest.java
+++ 
b/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopJobTrackerSelfTest.java
@@ -96,7 +96,9 @@ public class HadoopJobTrackerSelfTest extends 
HadoopAbstractSelfTest {
         HadoopConfiguration cfg = super.hadoopConfiguration(gridName);
 
         cfg.setMapReducePlanner(new HadoopTestRoundRobinMrPlanner());
-        cfg.setExternalExecution(false);
+
+        // TODO: IGNITE-404: Uncomment when fixed.
+        //cfg.setExternalExecution(false);
 
         return cfg;
     }

http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/dab66dc1/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopMapReduceEmbeddedSelfTest.java
----------------------------------------------------------------------
diff --git 
a/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopMapReduceEmbeddedSelfTest.java
 
b/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopMapReduceEmbeddedSelfTest.java
index 4a6e1ef..a329a95 100644
--- 
a/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopMapReduceEmbeddedSelfTest.java
+++ 
b/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopMapReduceEmbeddedSelfTest.java
@@ -46,7 +46,8 @@ public class HadoopMapReduceEmbeddedSelfTest extends 
HadoopMapReduceTest {
     @Override public HadoopConfiguration hadoopConfiguration(String gridName) {
         HadoopConfiguration cfg = super.hadoopConfiguration(gridName);
 
-        cfg.setExternalExecution(false);
+        // TODO: IGNITE-404: Uncomment when fixed.
+        //cfg.setExternalExecution(false);
 
         return cfg;
     }

http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/dab66dc1/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopSortingExternalTest.java
----------------------------------------------------------------------
diff --git 
a/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopSortingExternalTest.java
 
b/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopSortingExternalTest.java
index 772e77d..05c90e3 100644
--- 
a/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopSortingExternalTest.java
+++ 
b/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopSortingExternalTest.java
@@ -27,7 +27,8 @@ public class HadoopSortingExternalTest extends 
HadoopSortingTest {
     @Override public HadoopConfiguration hadoopConfiguration(String gridName) {
         HadoopConfiguration cfg = super.hadoopConfiguration(gridName);
 
-        cfg.setExternalExecution(true);
+        // TODO: IGNITE-404: Uncomment when fixed.
+        //cfg.setExternalExecution(true);
 
         return cfg;
     }

http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/dab66dc1/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopSortingTest.java
----------------------------------------------------------------------
diff --git 
a/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopSortingTest.java
 
b/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopSortingTest.java
index 3f6594a..1005de0 100644
--- 
a/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopSortingTest.java
+++ 
b/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopSortingTest.java
@@ -68,7 +68,8 @@ public class HadoopSortingTest extends HadoopAbstractSelfTest 
{
     @Override public HadoopConfiguration hadoopConfiguration(String gridName) {
         HadoopConfiguration cfg = super.hadoopConfiguration(gridName);
 
-        cfg.setExternalExecution(false);
+        // TODO: IGNITE-404: Uncomment when fixed.
+        //cfg.setExternalExecution(false);
 
         return cfg;
     }

http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/dab66dc1/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopTaskExecutionSelfTest.java
----------------------------------------------------------------------
diff --git 
a/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopTaskExecutionSelfTest.java
 
b/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopTaskExecutionSelfTest.java
index 20c5db2..0107907 100644
--- 
a/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopTaskExecutionSelfTest.java
+++ 
b/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/HadoopTaskExecutionSelfTest.java
@@ -109,7 +109,9 @@ public class HadoopTaskExecutionSelfTest extends 
HadoopAbstractSelfTest {
         HadoopConfiguration cfg = super.hadoopConfiguration(gridName);
 
         cfg.setMaxParallelTasks(5);
-        cfg.setExternalExecution(false);
+
+        // TODO: IGNITE-404: Uncomment when fixed.
+        //cfg.setExternalExecution(false);
 
         return cfg;
     }

http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/dab66dc1/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/taskexecutor/external/HadoopExternalTaskExecutionSelfTest.java
----------------------------------------------------------------------
diff --git 
a/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/taskexecutor/external/HadoopExternalTaskExecutionSelfTest.java
 
b/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/taskexecutor/external/HadoopExternalTaskExecutionSelfTest.java
index 59ac445..15f16d9 100644
--- 
a/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/taskexecutor/external/HadoopExternalTaskExecutionSelfTest.java
+++ 
b/modules/hadoop/src/test/java/org/apache/ignite/internal/processors/hadoop/taskexecutor/external/HadoopExternalTaskExecutionSelfTest.java
@@ -58,7 +58,8 @@ public class HadoopExternalTaskExecutionSelfTest extends 
HadoopAbstractSelfTest
     @Override public HadoopConfiguration hadoopConfiguration(String gridName) {
         HadoopConfiguration cfg = super.hadoopConfiguration(gridName);
 
-        cfg.setExternalExecution(true);
+        // TODO: IGNITE-404: Uncomment when fixed.
+        //cfg.setExternalExecution(true);
 
         return cfg;
     }

http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/dab66dc1/modules/hadoop/src/test/java/org/apache/ignite/testsuites/IgniteHadoopTestSuite.java
----------------------------------------------------------------------
diff --git 
a/modules/hadoop/src/test/java/org/apache/ignite/testsuites/IgniteHadoopTestSuite.java
 
b/modules/hadoop/src/test/java/org/apache/ignite/testsuites/IgniteHadoopTestSuite.java
index 4790e63..4d2a4b6 100644
--- 
a/modules/hadoop/src/test/java/org/apache/ignite/testsuites/IgniteHadoopTestSuite.java
+++ 
b/modules/hadoop/src/test/java/org/apache/ignite/testsuites/IgniteHadoopTestSuite.java
@@ -106,11 +106,11 @@ public class IgniteHadoopTestSuite extends TestSuite {
 
         suite.addTest(new 
TestSuite(ldr.loadClass(HadoopMapReduceEmbeddedSelfTest.class.getName())));
 
-        suite.addTest(new 
TestSuite(ldr.loadClass(HadoopExternalTaskExecutionSelfTest.class.getName())));
-        suite.addTest(new 
TestSuite(ldr.loadClass(HadoopExternalCommunicationSelfTest.class.getName())));
-
         suite.addTest(new 
TestSuite(ldr.loadClass(HadoopSortingTest.class.getName())));
 
+        // TODO: IGNITE-404: Uncomment when fixed.
+        //suite.addTest(new 
TestSuite(ldr.loadClass(HadoopExternalTaskExecutionSelfTest.class.getName())));
+        //suite.addTest(new 
TestSuite(ldr.loadClass(HadoopExternalCommunicationSelfTest.class.getName())));
         suite.addTest(new 
TestSuite(ldr.loadClass(HadoopSortingExternalTest.class.getName())));
 
         suite.addTest(new 
TestSuite(ldr.loadClass(HadoopGroupingTest.class.getName())));

Reply via email to