Skip to content

Commit 6184c07

Browse files
author
Marcelo Vanzin
committed
Rename field.
1 parent 4c19196 commit 6184c07

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

launcher/src/main/java/org/apache/spark/launcher/SparkSubmitCommandBuilder.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ class SparkSubmitCommandBuilder extends AbstractCommandBuilder {
7474
* to parse the command lines for things like bin/spark-shell, which allows users to mix and
7575
* match arguments (e.g. "bin/spark-shell SparkShellArg --master foo").
7676
*/
77-
private boolean isSpecialClass;
77+
private boolean allowsMixedArguments;
7878

7979
SparkSubmitCommandBuilder() {
8080
this.sparkArgs = new ArrayList<String>();
@@ -84,11 +84,11 @@ class SparkSubmitCommandBuilder extends AbstractCommandBuilder {
8484
this();
8585
List<String> submitArgs = args;
8686
if (args.size() > 0 && args.get(0).equals(PYSPARK_SHELL)) {
87-
this.isSpecialClass = true;
87+
this.allowsMixedArguments = true;
8888
appResource = PYSPARK_SHELL_RESOURCE;
8989
submitArgs = args.subList(1, args.size());
9090
} else {
91-
this.isSpecialClass = false;
91+
this.allowsMixedArguments = false;
9292
}
9393

9494
new OptionParser().parse(submitArgs);
@@ -287,7 +287,7 @@ protected boolean handle(String opt, String value) {
287287
// non-spark-submit arguments.
288288
mainClass = value;
289289
if (specialClasses.containsKey(value)) {
290-
isSpecialClass = true;
290+
allowsMixedArguments = true;
291291
appResource = specialClasses.get(value);
292292
}
293293
} else {
@@ -303,7 +303,7 @@ protected boolean handleUnknown(String opt) {
303303
// normal mode, any unrecognized parameter triggers the end of command line parsing, and the
304304
// parameter itself will be interpreted by SparkSubmit as the application resource. The
305305
// remaining params will be appended to the list of SparkSubmit arguments.
306-
if (isSpecialClass) {
306+
if (allowsMixedArguments) {
307307
appArgs.add(opt);
308308
return true;
309309
} else {

0 commit comments

Comments
 (0)