[GitHub] spark pull request #18499: [SPARK-21176][WEB UI] Use a single ProxyServlet t...

2017-07-01 Thread aosagie
GitHub user aosagie opened a pull request: https://github.com/apache/spark/pull/18499 [SPARK-21176][WEB UI] Use a single ProxyServlet to proxy all workers and applications ## What changes were proposed in this pull request? Currently, each application and each worker

[GitHub] spark issue #18499: [SPARK-21176][WEB UI] Use a single ProxyServlet to proxy...

2017-07-02 Thread aosagie
Github user aosagie commented on the issue: https://github.com/apache/spark/pull/18499 Hi @cloud-fan, Any reason the test results didn't get posted? Is there some step I need to take? --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] spark pull request #18499: [SPARK-21176][WEB UI] Use a single ProxyServlet t...

2017-07-04 Thread aosagie
Github user aosagie commented on a diff in the pull request: https://github.com/apache/spark/pull/18499#discussion_r125512070 --- Diff: core/src/main/scala/org/apache/spark/ui/JettyUtils.scala --- @@ -194,30 +194,26 @@ private[spark] object JettyUtils extends Logging

[GitHub] spark issue #18499: [SPARK-21176][WEB UI] Use a single ProxyServlet to proxy...

2017-07-10 Thread aosagie
Github user aosagie commented on the issue: https://github.com/apache/spark/pull/18499 @cloud-fan Hi, I just pushed a change to up the selector threads from 1 to 8. Can I get a retest please? @jiangxb1987 Sorry to bother, but is there anyone available to give guidance on

[GitHub] spark issue #18499: [SPARK-21176][WEB UI] Use a single ProxyServlet to proxy...

2017-08-01 Thread aosagie
Github user aosagie commented on the issue: https://github.com/apache/spark/pull/18499 Hey @ajbozarth. Any chance you could provide a review or some guidance on anything I can do to make this PR more amenable? --- If your project is set up for it, you can reply to this email and

[GitHub] spark issue #18499: [SPARK-21176][WEB UI] Use a single ProxyServlet to proxy...

2017-08-10 Thread aosagie
Github user aosagie commented on the issue: https://github.com/apache/spark/pull/18499 Yes, I agree with @IngoSchuster --- 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

[GitHub] spark pull request #18915: [SPARK-21176][WEB UI] Format worker page links to...

2017-08-10 Thread aosagie
GitHub user aosagie opened a pull request: https://github.com/apache/spark/pull/18915 [SPARK-21176][WEB UI] Format worker page links to work with proxy ## What changes were proposed in this pull request? Several links on the worker page do not work correctly with the proxy

[GitHub] spark issue #18915: [SPARK-21176][WEB UI] Format worker page links to work w...

2017-08-10 Thread aosagie
Github user aosagie commented on the issue: https://github.com/apache/spark/pull/18915 Hey @cloud-fan and @ajbozarth Thanks for checking my last PR. Any chance I could get one more look at this clean up? Some of the existing link handling was different between the Applications

[GitHub] spark issue #18915: [SPARK-21176][WEB UI] Format worker page links to work w...

2017-08-12 Thread aosagie
Github user aosagie commented on the issue: https://github.com/apache/spark/pull/18915 Here's a screenshot of the worker page link changes when the **proxy is off**: ![screenshot from 2017-08-12 04-07-14](https://user-images.githubusercontent.com/1888108/29239160-29528df6