Updated comment based on feedback.

harishreedharan · harishreedharan · commit b3f9b9d52dac · 2015-05-21T12:27:56.000-07:00
diff --git a/yarn/src/main/scala/org/apache/spark/scheduler/cluster/YarnClusterSchedulerBackend.scala b/yarn/src/main/scala/org/apache/spark/scheduler/cluster/YarnClusterSchedulerBackend.scala
@@ -98,7 +98,12 @@ private[spark] class YarnClusterSchedulerBackend(
             }
           }
 
-        // Build the HTTP address for the node and build the URL for the logs.
+        // Now that we have found the report for the Node Manager that the AM is running on, we
+        // can get the base HTTP address for the Node manager from the report.
+        // The format used for the logs for each container is well-known and can be constructed
+        // using the NM's HTTP address and the container ID.
+        // The NM may be running several containers, but we can build the URL for the AM using
+        // the AM's container ID, which we already know.
         nodeReport.foreach { report =>
           val httpAddress = report.getHttpAddress
           // lookup appropriate http scheme for container log urls

Original file line number	Diff line number	Diff line change
`@@ -98,7 +98,12 @@ private[spark] class YarnClusterSchedulerBackend(`
`98`	`98`	`}`
`99`	`99`	`}`
`100`	`100`
`101`		`- // Build the HTTP address for the node and build the URL for the logs.`
	`101`	`+ // Now that we have found the report for the Node Manager that the AM is running on, we`
	`102`	`+ // can get the base HTTP address for the Node manager from the report.`
	`103`	`+ // The format used for the logs for each container is well-known and can be constructed`
	`104`	`+ // using the NM's HTTP address and the container ID.`
	`105`	`+ // The NM may be running several containers, but we can build the URL for the AM using`
	`106`	`+ // the AM's container ID, which we already know.`
`102`	`107`	`nodeReport.foreach { report =>`
`103`	`108`	`val httpAddress = report.getHttpAddress`
`104`	`109`	`// lookup appropriate http scheme for container log urls`