Repository: spark
Updated Branches:
  refs/heads/branch-1.1 2c4117083 -> 35875e9ec


[SPARK-3067] JobProgressPage could not show Fair Scheduler Pools section 
sometimes

JobProgressPage could not show Fair Scheduler Pools section sometimes.
SparkContext starts webui and then postEnvironmentUpdate. Sometimes 
JobProgressPage is accessed between webui starting and postEnvironmentUpdate, 
then the lazy val isFairScheduler will be false. The Fair Scheduler Pools 
section will not display any more.

Author: yantangzhai <[email protected]>
Author: YanTangZhai <[email protected]>

Closes #1966 from YanTangZhai/SPARK-3067 and squashes the following commits:

d4323f8 [yantangzhai] update [SPARK-3067] JobProgressPage could not show Fair 
Scheduler Pools section sometimes
8a00106 [YanTangZhai] Merge pull request #6 from apache/master
b6391cc [yantangzhai] revert [SPARK-3067] JobProgressPage could not show Fair 
Scheduler Pools section sometimes
d2226cd [yantangzhai] [SPARK-3067] JobProgressPage could not show Fair 
Scheduler Pools section sometimes
cbcba66 [YanTangZhai] Merge pull request #3 from apache/master
aac7f7b [yantangzhai] [SPARK-3067] JobProgressPage could not show Fair 
Scheduler Pools section sometimes
cdef539 [YanTangZhai] Merge pull request #1 from apache/master

(cherry picked from commit dedace83f35cba0f833d962acbd75572318948c4)
Signed-off-by: Andrew Or <[email protected]>


Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/35875e9e
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/35875e9e
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/35875e9e

Branch: refs/heads/branch-1.1
Commit: 35875e9ec6e63a8f28a0bdc66f83d9b623bf02bb
Parents: 2c41170
Author: yantangzhai <[email protected]>
Authored: Thu Oct 16 19:25:37 2014 -0700
Committer: Andrew Or <[email protected]>
Committed: Thu Oct 16 19:27:32 2014 -0700

----------------------------------------------------------------------
 core/src/main/scala/org/apache/spark/SparkContext.scala | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/35875e9e/core/src/main/scala/org/apache/spark/SparkContext.scala
----------------------------------------------------------------------
diff --git a/core/src/main/scala/org/apache/spark/SparkContext.scala 
b/core/src/main/scala/org/apache/spark/SparkContext.scala
index 0e10d6c..5ec86eb 100644
--- a/core/src/main/scala/org/apache/spark/SparkContext.scala
+++ b/core/src/main/scala/org/apache/spark/SparkContext.scala
@@ -228,7 +228,6 @@ class SparkContext(config: SparkConf) extends Logging {
       // For tests, do not enable the UI
       None
     }
-  ui.foreach(_.bind())
 
   /** A default Hadoop Configuration for the Hadoop code (e.g. file systems) 
that we reuse. */
   val hadoopConfiguration: Configuration = {
@@ -341,6 +340,10 @@ class SparkContext(config: SparkConf) extends Logging {
   postEnvironmentUpdate()
   postApplicationStart()
 
+  // Bind the SparkUI after starting the task scheduler
+  // because certain pages and listeners depend on it
+  ui.foreach(_.bind())
+
   private[spark] var checkpointDir: Option[String] = None
 
   // Thread Local variable that can be used by users to pass information down 
the stack


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to