Skip to content

Commit

Permalink
[MINOR] Add requestHeaderSize debug log
Browse files Browse the repository at this point in the history
## 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 <gabor.g.somogyi@gmail.com>
Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
(cherry picked from commit 0b6c2c2)
Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
  • Loading branch information
gaborgsomogyi authored and kai-chi committed Sep 26, 2019
1 parent 8f7c4a8 commit e8bc57f
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion core/src/main/scala/org/apache/spark/ui/JettyUtils.scala
Original file line number Diff line number Diff line change
Expand Up @@ -357,7 +357,9 @@ private[spark] object JettyUtils extends Logging {
(connector, connector.getLocalPort())
}
val httpConfig = new HttpConfiguration()
httpConfig.setRequestHeaderSize(conf.get(UI_REQUEST_HEADER_SIZE).toInt)
val requestHeaderSize = conf.get(UI_REQUEST_HEADER_SIZE).toInt
logDebug(s"Using requestHeaderSize: $requestHeaderSize")
httpConfig.setRequestHeaderSize(requestHeaderSize)

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

0 comments on commit e8bc57f

Please sign in to comment.