Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/290#issuecomment-40173436
This is a really nice re-factoring! Unfortunately I just had time to look
at some of it due to running into a weird performance issue. For some reason
the pages were taking a long time to load (e.g. several seconds). I did some
profiling and it seemed like there was a code path when serving the static data
where it would look for gzipped versions of the static files and I don't think
this lookup was cached, so it was having to walk through the jar.
I fixed that issue by just disabling that feature for the static servlets:
```
def createStaticHandler(resourceBase: String, path: String):
ServletContextHandler = {
val contextHandler = new ServletContextHandler
+
contextHandler.setInitParameter("org.eclipse.jetty.servlet.Default.gzip",
"false")
val staticHandler = new DefaultServlet
val holder = new ServletHolder(staticHandler)
Option(getClass.getClassLoader.getResource(resourceBase)) match {
```
---
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.
---
