Apache
/
hadoop
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

YARN-8997. [Submarine] Small refactors of modifier, condition check and redundant local variables. Contributed by Zhankun Tang.

This commit is contained in:
Giovanni Matteo Fumarola 2018-11-12 15:06:43 -08:00
parent f944f33832
commit e269c3fb5a
5 changed files with 6 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-submarine/src/main/java/org/apache/hadoop/yarn/submarine/client/cli/CliUtils.java
View File

@ -156,9 +156,7 @@ public class CliUtils {
return true;
if (args.length == 1) {
if (args[0].equals("-h") || args[0].equals("--help")) {
return true;
}
return args[0].equals("-h") || args[0].equals("--help");
}
return false;

6
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-submarine/src/main/java/org/apache/hadoop/yarn/submarine/runtimes/common/FSBasedSubmarineStorageImpl.java
View File

@ -73,8 +73,7 @@ public class FSBasedSubmarineStorageImpl extends SubmarineStorage {
private Path getModelInfoPath(String modelName, String version, boolean create)
throws IOException {
Path modelDir = rdm.getModelDir(modelName, create);
Path modelInfo = new Path(modelDir, version + ".info");
return modelInfo;
return new Path(modelDir, version + ".info");
}
private void serializeMap(FSDataOutputStream fos, Map<String, String> map)
@ -98,7 +97,6 @@ public class FSBasedSubmarineStorageImpl extends SubmarineStorage {
private Path getJobInfoPath(String jobName, boolean create) throws IOException {
Path path = rdm.getJobStagingArea(jobName, create);
Path fileName = new Path(path, "job.info");
return fileName;
return new Path(path, "job.info");
}
}

3
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-submarine/src/main/java/org/apache/hadoop/yarn/submarine/runtimes/yarnservice/YarnServiceJobMonitor.java
View File

@ -40,7 +40,6 @@ public class YarnServiceJobMonitor extends JobMonitor {
}
Service serviceSpec = this.serviceClient.getStatus(jobName);
JobStatus jobStatus = JobStatusBuilder.fromServiceSpec(serviceSpec);
return jobStatus;
return JobStatusBuilder.fromServiceSpec(serviceSpec);
}
}

5
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-submarine/src/main/java/org/apache/hadoop/yarn/submarine/runtimes/yarnservice/YarnServiceJobSubmitter.java
View File

@ -103,10 +103,7 @@ public class YarnServiceJobSubmitter implements JobSubmitter {
}
private boolean needHdfs(String content) {
if (content != null && content.contains("hdfs://")) {
return true;
}
return false;
return content != null && content.contains("hdfs://");
}
private void addHdfsClassPathIfNeeded(RunJobParameters parameters,

4
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-submarine/src/main/java/org/apache/hadoop/yarn/submarine/runtimes/yarnservice/YarnServiceUtils.java
View File

@ -58,9 +58,7 @@ public class YarnServiceUtils {
private static String getDNSNameCommonSuffix(String serviceName,
String userName, String domain, int port) {
String commonEndpointSuffix =
"." + serviceName + "." + userName + "." + domain + ":" + port;
return commonEndpointSuffix;
return "." + serviceName + "." + userName + "." + domain + ":" + port;
}
public static String getTFConfigEnv(String curCommponentName, int nWorkers,