Skip to content

Commit ed0c57e

Browse files
shahidki31srowen
authored andcommitted
[SPARK-25861][MINOR][WEBUI] Remove unused refreshInterval parameter from the headerSparkPage method.
## What changes were proposed in this pull request? 'refreshInterval' is not used any where in the headerSparkPage method. So, we don't need to pass the parameter while calling the 'headerSparkPage' method. ## How was this patch tested? Existing tests Closes #22864 from shahidki31/unusedCode. Authored-by: Shahid <shahidki31@gmail.com> Signed-off-by: Sean Owen <sean.owen@databricks.com>
1 parent 3404a73 commit ed0c57e

File tree

6 files changed

+5
-6
lines changed

6 files changed

+5
-6
lines changed

core/src/main/scala/org/apache/spark/ui/UIUtils.scala

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -218,7 +218,6 @@ private[spark] object UIUtils extends Logging {
218218
title: String,
219219
content: => Seq[Node],
220220
activeTab: SparkUITab,
221-
refreshInterval: Option[Int] = None,
222221
helpText: Option[String] = None,
223222
showVisualization: Boolean = false,
224223
useDataTables: Boolean = false): Seq[Node] = {

sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ private[ui] class AllExecutionsPage(parent: SQLTab) extends WebUIPage("") with L
146146
</ul>
147147
</div>
148148

149-
UIUtils.headerSparkPage(request, "SQL", summary ++ content, parent, Some(5000))
149+
UIUtils.headerSparkPage(request, "SQL", summary ++ content, parent)
150150
}
151151

152152
private def executionsTable(

sql/core/src/main/scala/org/apache/spark/sql/execution/ui/ExecutionPage.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ class ExecutionPage(parent: SQLTab) extends WebUIPage("execution") with Logging
8484
}
8585

8686
UIUtils.headerSparkPage(
87-
request, s"Details for Query $executionId", content, parent, Some(5000))
87+
request, s"Details for Query $executionId", content, parent)
8888
}
8989

9090

sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/ui/ThriftServerPage.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ private[ui] class ThriftServerPage(parent: ThriftServerTab) extends WebUIPage(""
5050
generateSessionStatsTable(request) ++
5151
generateSQLStatsTable(request)
5252
}
53-
UIUtils.headerSparkPage(request, "JDBC/ODBC Server", content, parent, Some(5000))
53+
UIUtils.headerSparkPage(request, "JDBC/ODBC Server", content, parent)
5454
}
5555

5656
/** Generate basic stats of the thrift server program */

sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/ui/ThriftServerSessionPage.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ private[ui] class ThriftServerSessionPage(parent: ThriftServerTab)
5858
</h4> ++
5959
generateSQLStatsTable(request, sessionStat.sessionId)
6060
}
61-
UIUtils.headerSparkPage(request, "JDBC/ODBC Session", content, parent, Some(5000))
61+
UIUtils.headerSparkPage(request, "JDBC/ODBC Session", content, parent)
6262
}
6363

6464
/** Generate basic stats of the thrift server program */

streaming/src/main/scala/org/apache/spark/streaming/ui/StreamingPage.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,7 @@ private[ui] class StreamingPage(parent: StreamingTab)
156156
generateStatTable() ++
157157
generateBatchListTables()
158158
}
159-
SparkUIUtils.headerSparkPage(request, "Streaming Statistics", content, parent, Some(5000))
159+
SparkUIUtils.headerSparkPage(request, "Streaming Statistics", content, parent)
160160
}
161161

162162
/**

0 commit comments

Comments
 (0)