Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/21688#discussion_r219251713
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala ---
@@ -278,7 +198,7 @@ private[ui] class StagePage(parent: StagesTab, store:
AppStatusStore) extends We
}
}
val currentTime = System.currentTimeMillis()
- val (taskTable, taskTableHTML) = try {
+ val taskTable = try {
val _taskTable = new TaskPagedTable(
--- End diff --
seems like this shouldn't be needed anymore. Feels like a bunch of stuff
in here should be able to be removed now?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
