This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new 0b6c2c2  [MINOR] Add requestHeaderSize debug log
0b6c2c2 is described below

commit 0b6c2c259c1ed109a824b678c9ccbd9fd767d2fe
Author: Gabor Somogyi <[email protected]>
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 <[email protected]>
    Signed-off-by: Dongjoon Hyun <[email protected]>
---
 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 e04eb84..ff7baf4 100644
--- a/core/src/main/scala/org/apache/spark/ui/JettyUtils.scala
+++ b/core/src/main/scala/org/apache/spark/ui/JettyUtils.scala
@@ -297,7 +297,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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to