Details
-
Improvement
-
Status: Resolved
-
Minor
-
Resolution: Fixed
-
None
-
None
-
None
Description
In YarnServiceJobSubmitter#needHdfs. Below code can be simplified to just one line.
if (content != null && content.contains("hdfs://")) { return true; } return false;
return content != null && content.contains("hdfs://");
In CliUtils#argsForHelp
if (args[0].equals("-h") || args[0].equals("--help")) { return true; }
Can be simlified to:
return args[0].equals("-h") || args[0].equals("--help");
And several redundant variables can be removed.