You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by do...@apache.org on 2019/07/03 16:35:05 UTC
[spark] branch branch-2.4 updated: [MINOR] Add requestHeaderSize
debug log
This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch branch-2.4
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/branch-2.4 by this push:
new ec6d0c9 [MINOR] Add requestHeaderSize debug log
ec6d0c9 is described below
commit ec6d0c9ab2a3cab586e73def9167a120780a2fbe
Author: Gabor Somogyi <ga...@gmail.com>
AuthorDate: Wed Jul 3 09:34:31 2019 -0700
[MINOR] Add requestHeaderSize debug log
## What changes were proposed in this pull request?
`requestHeaderSize` is added in https://github.com/apache/spark/pull/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 #25045 from gaborgsomogyi/SPARK-26118.
Authored-by: Gabor Somogyi <ga...@gmail.com>
Signed-off-by: Dongjoon Hyun <dh...@apple.com>
(cherry picked from commit 0b6c2c259c1ed109a824b678c9ccbd9fd767d2fe)
Signed-off-by: Dongjoon Hyun <dh...@apple.com>
---
core/src/main/scala/org/apache/spark/ui/JettyUtils.scala | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/core/src/main/scala/org/apache/spark/ui/JettyUtils.scala b/core/src/main/scala/org/apache/spark/ui/JettyUtils.scala
index 316af9b..4a168cf 100644
--- a/core/src/main/scala/org/apache/spark/ui/JettyUtils.scala
+++ b/core/src/main/scala/org/apache/spark/ui/JettyUtils.scala
@@ -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 =>
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org