Commit 4cdc327d authored by wenmo's avatar wenmo

[Fix-556][adimin,gateway] Fix flinkLibPath in K8S configuration is null

parent 9c06fc27
...@@ -53,15 +53,13 @@ public class ClusterConfigurationServiceImpl extends SuperServiceImpl<ClusterCon ...@@ -53,15 +53,13 @@ public class ClusterConfigurationServiceImpl extends SuperServiceImpl<ClusterCon
config.get("flinkLibPath").toString(), config.get("flinkLibPath").toString(),
config.get("hadoopConfigPath").toString())); config.get("hadoopConfigPath").toString()));
} else { } else {
gatewayConfig.setClusterConfig(ClusterConfig.build(config.get("flinkConfigPath").toString(), gatewayConfig.setClusterConfig(ClusterConfig.build(config.get("flinkConfigPath").toString()));
config.get("flinkLibPath").toString(),
""));
} }
if (config.containsKey("flinkConfig")) { if (config.containsKey("flinkConfig")) {
gatewayConfig.setFlinkConfig(FlinkConfig.build((Map<String, String>) config.get("flinkConfig"))); gatewayConfig.setFlinkConfig(FlinkConfig.build((Map<String, String>) config.get("flinkConfig")));
} }
if (Asserts.isEqualsIgnoreCase(clusterConfiguration.getType(), "Yarn")) { if (Asserts.isEqualsIgnoreCase(clusterConfiguration.getType(), "Yarn")) {
gatewayConfig.setType(GatewayType.YARN_PER_JOB); gatewayConfig.setType(GatewayType.YARN_APPLICATION);
} else if (Asserts.isEqualsIgnoreCase(clusterConfiguration.getType(), "Kubernetes")) { } else if (Asserts.isEqualsIgnoreCase(clusterConfiguration.getType(), "Kubernetes")) {
gatewayConfig.setType(GatewayType.KUBERNETES_APPLICATION); gatewayConfig.setType(GatewayType.KUBERNETES_APPLICATION);
Map kubernetesConfig = (Map) config.get("kubernetesConfig"); Map kubernetesConfig = (Map) config.get("kubernetesConfig");
......
...@@ -20,12 +20,20 @@ public class ClusterConfig { ...@@ -20,12 +20,20 @@ public class ClusterConfig {
public ClusterConfig() { public ClusterConfig() {
} }
public ClusterConfig(String flinkConfigPath) {
this.flinkConfigPath = flinkConfigPath;
}
public ClusterConfig(String flinkConfigPath, String flinkLibPath, String yarnConfigPath) { public ClusterConfig(String flinkConfigPath, String flinkLibPath, String yarnConfigPath) {
this.flinkConfigPath = flinkConfigPath; this.flinkConfigPath = flinkConfigPath;
this.flinkLibPath = flinkLibPath; this.flinkLibPath = flinkLibPath;
this.yarnConfigPath = yarnConfigPath; this.yarnConfigPath = yarnConfigPath;
} }
public static ClusterConfig build(String flinkConfigPath) {
return new ClusterConfig(flinkConfigPath);
}
public static ClusterConfig build(String flinkConfigPath, String flinkLibPath, String yarnConfigPath) { public static ClusterConfig build(String flinkConfigPath, String flinkLibPath, String yarnConfigPath) {
return new ClusterConfig(flinkConfigPath, flinkLibPath, yarnConfigPath); return new ClusterConfig(flinkConfigPath, flinkLibPath, yarnConfigPath);
} }
......
...@@ -131,11 +131,6 @@ public abstract class KubernetesGateway extends AbstractGateway { ...@@ -131,11 +131,6 @@ public abstract class KubernetesGateway extends AbstractGateway {
if (Asserts.isNull(config.getFlinkConfig().getJobId())) { if (Asserts.isNull(config.getFlinkConfig().getJobId())) {
throw new GatewayException("No job id was specified. Please specify a job to which you would like to savepont."); throw new GatewayException("No job id was specified. Please specify a job to which you would like to savepont.");
} }
if (Asserts.isNotNullString(config.getClusterConfig().getYarnConfigPath())) {
configuration = GlobalConfiguration.loadConfiguration(config.getClusterConfig().getYarnConfigPath());
} else {
configuration = new Configuration();
}
SavePointResult result = SavePointResult.build(getType()); SavePointResult result = SavePointResult.build(getType());
configuration.set(KubernetesConfigOptions.CLUSTER_ID, config.getClusterConfig().getAppId()); configuration.set(KubernetesConfigOptions.CLUSTER_ID, config.getClusterConfig().getAppId());
KubernetesClusterClientFactory clusterClientFactory = new KubernetesClusterClientFactory(); KubernetesClusterClientFactory clusterClientFactory = new KubernetesClusterClientFactory();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment