[hotfix] Clean up CliFrontend after removing web client
Project: http://git-wip-us.apache.org/repos/asf/flink/repo
Commit: http://git-wip-us.apache.org/repos/asf/flink/commit/17fa6a9b
Tree: http://git-wip-us.apache.org/repos/asf/flink/tree/17fa6a9b
Diff: http://git-wip-us.apache.org/repos/asf/flink/diff/17fa6a9b
Branch: refs/heads/master
Commit: 17fa6a9bc965eb4dcd64123b7d1c66a75c077db6
Parents: c0fd36b
Author: Stephan Ewen
Authored: Fri Jan 15 18:36:13 2016 +0100
Committer: Stephan Ewen
Committed: Sat Jan 16 15:46:56 2016 +0100
--
.../org/apache/flink/client/CliFrontend.java| 125 ---
.../flink/yarn/FlinkYarnSessionCliTest.java | 10 +-
.../apache/flink/yarn/FlinkYarnClientBase.java | 9 +-
.../flink/yarn/ApplicationMasterBase.scala | 8 +-
4 files changed, 62 insertions(+), 90 deletions(-)
--
http://git-wip-us.apache.org/repos/asf/flink/blob/17fa6a9b/flink-clients/src/main/java/org/apache/flink/client/CliFrontend.java
--
diff --git
a/flink-clients/src/main/java/org/apache/flink/client/CliFrontend.java
b/flink-clients/src/main/java/org/apache/flink/client/CliFrontend.java
index 7e1cef7..4b9bd06 100644
--- a/flink-clients/src/main/java/org/apache/flink/client/CliFrontend.java
+++ b/flink-clients/src/main/java/org/apache/flink/client/CliFrontend.java
@@ -19,13 +19,14 @@
package org.apache.flink.client;
import akka.actor.ActorSystem;
+
import org.apache.commons.cli.CommandLine;
+
import org.apache.flink.api.common.InvalidProgramException;
import org.apache.flink.api.common.JobExecutionResult;
import org.apache.flink.api.common.JobID;
import org.apache.flink.api.common.JobSubmissionResult;
import org.apache.flink.api.common.accumulators.AccumulatorHelper;
-import org.apache.flink.api.java.tuple.Tuple2;
import org.apache.flink.client.cli.CancelOptions;
import org.apache.flink.client.cli.CliArgsException;
import org.apache.flink.client.cli.CliFrontendParser;
@@ -65,8 +66,10 @@ import org.apache.flink.runtime.yarn.AbstractFlinkYarnClient;
import org.apache.flink.runtime.yarn.AbstractFlinkYarnCluster;
import org.apache.flink.runtime.yarn.FlinkYarnClusterStatus;
import org.apache.flink.util.StringUtils;
+
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
+
import scala.Some;
import scala.concurrent.Await;
import scala.concurrent.Future;
@@ -87,6 +90,7 @@ import java.util.Arrays;
import java.util.Collections;
import java.util.Comparator;
import java.util.Date;
+import java.util.HashMap;
import java.util.List;
import java.util.Map;
import java.util.Properties;
@@ -95,7 +99,6 @@ import java.util.concurrent.TimeUnit;
import static
org.apache.flink.runtime.messages.JobManagerMessages.DisposeSavepoint;
import static
org.apache.flink.runtime.messages.JobManagerMessages.DisposeSavepointFailure;
import static
org.apache.flink.runtime.messages.JobManagerMessages.TriggerSavepointFailure;
-import static
org.apache.flink.runtime.messages.JobManagerMessages.getRequestRunningJobsStatus;
/**
* Implementation of a simple command line frontend for executing programs.
@@ -133,6 +136,7 @@ public class CliFrontend {
private static final Logger LOG =
LoggerFactory.getLogger(CliFrontend.class);
+
private final Configuration config;
private final FiniteDuration askTimeout;
@@ -143,12 +147,6 @@ public class CliFrontend {
private AbstractFlinkYarnCluster yarnCluster;
- static boolean webFrontend = false;
-
- private FlinkPlan optimizedPlan;
-
- private PackagedProgram packagedProgram;
-
/**
*
* @throws Exception Thrown if the configuration directory was not
found, the configuration could not
@@ -222,9 +220,9 @@ public class CliFrontend {
// handle the YARN client's dynamic properties
String dynamicPropertiesEncoded =
yarnProperties.getProperty(YARN_PROPERTIES_DYNAMIC_PROPERTIES_STRING);
- List> dynamicProperties =
getDynamicProperties(dynamicPropertiesEncoded);
- for (Tuple2 dynamicProperty :
dynamicProperties) {
- this.config.setString(dynamicProperty.f0,
dynamicProperty.f1);
+ Map dynamicProperties =
getDynamicProperties(dynamicPropertiesEncoded);
+ for (Map.Entry dynamicProperty :
dynamicProperties.entrySet()) {
+ this.config.setString(dynamicProperty.getKey(),
dynamicProperty.getValue());
}
}
@@ -408,42 +406,34 @@ public class CliFrontend {
LOG.info("Creating program plan dump");