review comments

Author: tgravescs

docs/running-on-yarn.md

@@ -110,7 +110,7 @@ Most of the configs are the same for Spark on YARN as for other deployment modes
   <td><code>spark.yarn.access.namenodes</code></td>
   <td>(none)</td>
   <td>
-    A list of secure HDFS namenodes your spark application is going to access. For example, spark.yarn.access.namenodes=hdfs://nn1.com:8032,hdfs://nn2.com:8032. Spark acquires security Tokens for each of the namenodes so that the spark application can access those remote HDFS clusters.  
+    A list of secure HDFS namenodes your Spark application is going to access. For example, `spark.yarn.access.namenodes=hdfs://nn1.com:8032,hdfs://nn2.com:8032`. The Spark application must have acess to the namenodes listed and Kerberos must be properly configured to be able to access them (either in the same realm or in a trusted realm). Spark acquires security tokens for each of the namenodes so that the Spark application can access those remote HDFS clusters.
   </td>
 </tr>
 </table>

yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientBase.scala

@@ -601,7 +601,9 @@ object ClientBase extends Logging {
     YarnSparkHadoopUtil.addToEnvironment(env, Environment.CLASSPATH.name, path,
             File.pathSeparator)
 
-  // get the list of namenodes the user may access
+  /** 
+   * Get the list of namenodes the user may access.
+   */
   private[yarn] def getNameNodesToAccess(sparkConf: SparkConf): Set[Path] = {
     sparkConf.get("spark.yarn.access.namenodes", "").split(",").map(_.trim()).filter(!_.isEmpty)
       .map(new Path(_)).toSet
@@ -618,7 +620,9 @@ object ClientBase extends Logging {
     delegTokenRenewer
   }
 
-  // obtains tokens for the namenodes passed in and adds them to the credentials
+  /**
+   * Obtains tokens for the namenodes passed in and adds them to the credentials.
+   */
   private[yarn] def obtainTokensForNamenodes(paths: Set[Path], conf: Configuration,
     creds: Credentials) {
     if (UserGroupInformation.isSecurityEnabled()) {