Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/21450#discussion_r194915296 --- Diff: launcher/src/main/java/org/apache/spark/launcher/Main.java --- @@ -101,6 +99,22 @@ public static void main(String[] argsArray) throws Exception { } } + /** + * Prepare spark commands with the appropriate command builder. + * If printLaunchCommand is set then the commands will be printed to the stderr. + */ + private static List<String> buildCommand(AbstractCommandBuilder builder, --- End diff -- Please follow the Spark style for multi-line parameter lists.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org