Skip to content

Commit 0b6c2c2

Browse files
gaborgsomogyidongjoon-hyun
authored andcommitted
[MINOR] Add requestHeaderSize debug log
## What changes were proposed in this pull request? `requestHeaderSize` is added in apache#23090 and applies to Spark + History server UI as well. Without debug log it's hard to find out on which side what configuration is used. In this PR I've added a log message which prints out the value. ## How was this patch tested? Manually checked log files. Closes apache#25045 from gaborgsomogyi/SPARK-26118. Authored-by: Gabor Somogyi <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
1 parent 74f1176 commit 0b6c2c2

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

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

+3-1
Original file line numberDiff line numberDiff line change
@@ -297,7 +297,9 @@ private[spark] object JettyUtils extends Logging {
297297
(connector, connector.getLocalPort())
298298
}
299299
val httpConfig = new HttpConfiguration()
300-
httpConfig.setRequestHeaderSize(conf.get(UI_REQUEST_HEADER_SIZE).toInt)
300+
val requestHeaderSize = conf.get(UI_REQUEST_HEADER_SIZE).toInt
301+
logDebug(s"Using requestHeaderSize: $requestHeaderSize")
302+
httpConfig.setRequestHeaderSize(requestHeaderSize)
301303

302304
// If SSL is configured, create the secure connector first.
303305
val securePort = sslOptions.createJettySslContextFactory().map { factory =>

0 commit comments

Comments
 (0)