Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11615#discussion_r55743040
--- Diff: core/src/main/scala/org/apache/spark/ui/JettyUtils.scala ---
@@ -270,9 +270,19 @@ private[spark] object JettyUtils extends Logging {
gzipHandlers.foreach(collection.addHandler)
connectors.foreach(_.setHost(hostName))
+ // As each Acceptor will use one thread, the number of threads
should at least be the number
+ // of acceptors plus 1. (See SPARK-13776)
+ var minThreads = 1
+ connectors.collect { case c: AbstractConnector => c }.foreach { c =>
--- End diff --
Nit: the Scala math class is `math` vs `Math` but I wouldn't change it just
for that. I think this looks like the right kind of approach.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
