Javadoc updates.

Author: Marcelo Vanzin

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

@@ -62,28 +62,49 @@ protected AbstractLauncher(Map<String, String> env) {
   @SuppressWarnings("unchecked")
   private final T THIS = (T) this;
 
-  /** Set a custom JAVA_HOME for launching the Spark application. */
+  /**
+   * Set a custom JAVA_HOME for launching the Spark application.
+   *
+   * @param javaHome Path to the JAVA_HOME to use.
+   * @return This launcher.
+   */
   public T setJavaHome(String javaHome) {
     checkNotNull(javaHome, "javaHome");
     this.javaHome = javaHome;
     return THIS;
   }
 
-  /** Set a custom Spark installation location for the application. */
+  /**
+   * Set a custom Spark installation location for the application.
+   *
+   * @param sparkHome Path to the Spark installation to use.
+   * @return This launcher.
+   */
   public T setSparkHome(String sparkHome) {
     checkNotNull(sparkHome, "sparkHome");
     launcherEnv.put(ENV_SPARK_HOME, sparkHome);
     return THIS;
   }
 
-  /** Set a custom properties file with Spark configuration for the application. */
+  /**
+   * Set a custom properties file with Spark configuration for the application.
+   *
+   * @param path Path to custom properties file to use.
+   * @return This launcher.
+   */
   public T setPropertiesFile(String path) {
     checkNotNull(path, "path");
     this.propertiesFile = path;
     return THIS;
   }
 
-  /** Set a single configuration value for the application. */
+  /**
+   * Set a single configuration value for the application.
+   *
+   * @param key Configuration key.
+   * @param value The value to use.
+   * @return This launcher.
+   */
   public T setConf(String key, String value) {
     checkNotNull(key, "key");
     checkNotNull(value, "value");
@@ -117,7 +138,7 @@ List<String> buildShellCommand() throws IOException {
   }
 
   /**
-   * Loads the configuration file for the application, if it exists. This is  either the
+   * Loads the configuration file for the application, if it exists. This is either the
    * user-specified properties file, or the spark-defaults.conf file under the Spark configuration
    * directory.
    */

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

@@ -56,21 +56,36 @@ public SparkLauncher() {
     this.pyFiles = new ArrayList<String>();
   }
 
-  /** Set the application name. */
+  /**
+   * Set the application name.
+   *
+   * @param appName Application name.
+   * @return This launcher.
+   */
   public SparkLauncher setAppName(String appName) {
     checkNotNull(appName, "appName");
     this.appName = appName;
     return this;
   }
 
-  /** Set the Spark master for the application. */
+  /**
+   * Set the Spark master for the application.
+   *
+   * @param master Spark master.
+   * @return This launcher.
+   */
   public SparkLauncher setMaster(String master) {
     checkNotNull(master, "master");
     this.master = master;
     return this;
   }
 
-  /** Set the deploy mode for the application. */
+  /**
+   * Set the deploy mode for the application.
+   *
+   * @param mode Deploy mode.
+   * @return This launcher.
+   */
   public SparkLauncher setDeployMode(String mode) {
     checkNotNull(mode, "mode");
     this.deployMode = mode;
@@ -80,21 +95,34 @@ public SparkLauncher setDeployMode(String mode) {
   /**
    * Set the main application resource. This should be the location of a jar file for Scala/Java
    * applications, or a python script for PySpark applications.
+   *
+   * @param resource Path to the main application resource.
+   * @return This launcher.
    */
   public SparkLauncher setAppResource(String resource) {
     checkNotNull(resource, "resource");
     this.appResource = resource;
     return this;
   }
 
-  /** Sets the application class name for Java/Scala applications. */
+  /**
+   * Sets the application class name for Java/Scala applications.
+   *
+   * @param mainClass Application's main class.
+   * @return This launcher.
+   */
   public SparkLauncher setMainClass(String mainClass) {
     checkNotNull(mainClass, "mainClass");
     this.mainClass = mainClass;
     return this;
   }
 
-  /** Adds command line arguments for the application. */
+  /**
+   * Adds command line arguments for the application.
+   *
+   * @param args Arguments to pass to the application's main class.
+   * @return This launcher.
+   */
   public SparkLauncher addAppArgs(String... args) {
     for (String arg : args) {
       checkNotNull(arg, "arg");
@@ -103,28 +131,48 @@ public SparkLauncher addAppArgs(String... args) {
     return this;
   }
 
-  /** Adds a jar file to be submitted with the application. */
+  /**
+   * Adds a jar file to be submitted with the application.
+   *
+   * @param jar Path to the jar file.
+   * @return This launcher.
+   */
   public SparkLauncher addJar(String jar) {
     checkNotNull(jar, "jar");
     jars.add(jar);
     return this;
   }
 
-  /** Adds a file to be submitted with the application. */
+  /**
+   * Adds a file to be submitted with the application.
+   *
+   * @param file Path to the file.
+   * @return This launcher.
+   */
   public SparkLauncher addFile(String file) {
     checkNotNull(file, "file");
     files.add(file);
     return this;
   }
 
-  /** Adds a python file / zip / egg to be submitted with the application. */
+  /**
+   * Adds a python file / zip / egg to be submitted with the application.
+   *
+   * @param file Path to the file.
+   * @return This launcher.
+   */
   public SparkLauncher addPyFile(String file) {
     checkNotNull(file, "file");
     pyFiles.add(file);
     return this;
   }
 
-  /** Enables verbose reporting for SparkSubmit. */
+  /**
+   * Enables verbose reporting for SparkSubmit.
+   *
+   * @param verbose Whether to enable verbose output.
+   * @return This launcher.
+   */
   public SparkLauncher setVerbose(boolean verbose) {
     this.verbose = verbose;
     return this;