You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2021/05/14 01:55:39 UTC

[GitHub] [spark] srowen commented on a change in pull request #32539: [SPARK-35402][WebUI] Make thread pool of jetty server in Web UIs configurable

srowen commented on a change in pull request #32539:
URL: https://github.com/apache/spark/pull/32539#discussion_r632214185



##########
File path: core/src/main/scala/org/apache/spark/internal/config/UI.scala
##########
@@ -218,4 +218,13 @@ private[spark] object UI {
     .stringConf
     .transform(_.toUpperCase(Locale.ROOT))
     .createWithDefault("LOCAL")
+
+  val UI_THREADS = ConfigBuilder("spark.ui.threads")

Review comment:
       I would really rather not add yet another config. Can Jetty thread pools simply expand as needed up to some large max? that kind of resolves this automatically.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org