[SPARK-7932] Fix misleading scheduler delay visualization
The existing code rounds down to the nearest percent when computing the proportion of a task's time that was spent on each phase of execution, and then computes the scheduler delay proportion as 100 - sum(all other proportions). As a result, a few extra percent can end up in the scheduler delay. This commit eliminates the rounding so that the time visualizations correspond properly to the real times. sarutak If you could take a look at this, that would be great! Not sure if there's a good reason to round here that I missed. cc shivaram Author: Kay Ousterhout <kayousterhout@gmail.com> Closes #6484 from kayousterhout/SPARK-7932 and squashes the following commits: 1723cc4 [Kay Ousterhout] [SPARK-7932] Fix misleading scheduler delay visualization
This commit is contained in:
parent
834e699524
commit
04ddcd4db7
|
@ -527,7 +527,7 @@ private[ui] class StagePage(parent: StagesTab) extends WebUIPage("stage") {
|
|||
minLaunchTime = launchTime.min(minLaunchTime)
|
||||
maxFinishTime = finishTime.max(maxFinishTime)
|
||||
|
||||
def toProportion(time: Long) = (time.toDouble / totalExecutionTime * 100).toLong
|
||||
def toProportion(time: Long) = time.toDouble / totalExecutionTime * 100
|
||||
|
||||
val metricsOpt = taskUIData.taskMetrics
|
||||
val shuffleReadTime =
|
||||
|
|
Loading…
Reference in a new issue