[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114981588 Merged in master and branch-1.4 --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/6845 --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114570416 Merged build started. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114570332 Merged build triggered. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114569723 Here's the latest screenshot:  --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114570893 [Test build #35562 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35562/consoleFull) for PR 6845 at commit [`b63695f`](https://github.com/apache/spark/commit/b63695f7b993fe4b3aeaea3dbae4460aa7b886b2). --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114622105 Merged build finished. Test FAILed. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114622073 **[Test build #35562 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35562/console)** for PR 6845 at commit [`b63695f`](https://github.com/apache/spark/commit/b63695f7b993fe4b3aeaea3dbae4460aa7b886b2) after a configured wait of `175m`. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114649519 Merged build finished. Test PASSed. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114626933 Test this again. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114630573 [Test build #35582 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35582/consoleFull) for PR 6845 at commit [`b63695f`](https://github.com/apache/spark/commit/b63695f7b993fe4b3aeaea3dbae4460aa7b886b2). --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114649469 [Test build #35582 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35582/console) for PR 6845 at commit [`b63695f`](https://github.com/apache/spark/commit/b63695f7b993fe4b3aeaea3dbae4460aa7b886b2). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114628019 Please test this again. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114628484 test this please. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114630333 Merged build triggered. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114630360 Merged build started. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114384793 Test this again. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114446269 @zsxwing Thanks for spotting this, I will update the PR later today. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114423828 I just tried this PR. There is still one problem. The graphs in the inner table does not align with the outer ones.  Could you change `300px` to `350px` in this line https://github.com/apache/spark/blob/6ceb169608428a651d53c93bf73ca5ac53a6bde2/streaming/src/main/scala/org/apache/spark/streaming/ui/StreamingPage.scala#L313 and change `292px` to `342px` in this line https://github.com/apache/spark/blob/6ceb169608428a651d53c93bf73ca5ac53a6bde2/streaming/src/main/scala/org/apache/spark/streaming/ui/StreamingPage.scala#L459 This would fix the alignment issue. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114420915 Merged build started. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114430642 Merged build finished. Test FAILed. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114419986 Jenkins, retest this please. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114421335 [Test build #35540 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35540/consoleFull) for PR 6845 at commit [`eb610ee`](https://github.com/apache/spark/commit/eb610ee10371170a72259a207f71d06b687de2af). --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114430621 [Test build #35540 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35540/console) for PR 6845 at commit [`eb610ee`](https://github.com/apache/spark/commit/eb610ee10371170a72259a207f71d06b687de2af). * This patch **fails Spark unit tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114420859 Merged build triggered. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114683528 LGTM --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114208486 Merged build finished. Test FAILed. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114197572 I adjusted `#batches` so it's aligned with the ticks as well as the units on the timeline graph (`events/sec` and `sec`). --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114198954 [Test build #35464 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35464/consoleFull) for PR 6845 at commit [`dd46f98`](https://github.com/apache/spark/commit/dd46f98571d6f6a9bcb7c00a64b5cca226996b11). --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114228578 Merged build finished. Test FAILed. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114224831 Here's the final screenshot  --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114208463 [Test build #35464 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35464/console) for PR 6845 at commit [`dd46f98`](https://github.com/apache/spark/commit/dd46f98571d6f6a9bcb7c00a64b5cca226996b11). * This patch **fails Spark unit tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114197821 Merged build started. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-114197781 Merged build triggered. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113894430 I have this version which aligns with the units of the lineplot ```events/sec```:  Or do you want to align ```#batches``` with the ticks (10, 20, 30, ...)? --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113894728 Merged build started. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113899929 Merged build finished. Test FAILed. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113894632 Merged build triggered. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113899887 [Test build #35390 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35390/console) for PR 6845 at commit [`0564b62`](https://github.com/apache/spark/commit/0564b6279c7cbb17b91f95db4081342424d6e227). * This patch **fails Spark unit tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113894842 [Test build #35390 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35390/consoleFull) for PR 6845 at commit [`0564b62`](https://github.com/apache/spark/commit/0564b6279c7cbb17b91f95db4081342424d6e227). --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113978964 Yeah, I wanted it aligned with 10, 20, 30 ... all the letters on same line. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113746495 @tdas I'll see what I can do. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113785985 Do you mean above the axis? No. Still overlap. But `#batches` is on the top. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113783603 Do you mean above the axis? --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113786509 Mmh, in this case I think we won't be able to see ```#batches``` written since the contrast won't be high enough between gray and dark blue. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113783361 Looks a lot of space on the right. Could you just move `#batches` above the blue bar? --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113786756 Maybe we should put the axis on the bottom and ```#batches``` under the axis. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113768928 @tdas Here is the before:  And, here is the after:  --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113820716 I dont mind the extra white space at the right. Rather what is bugging me is the batch is not really aligned with x axis / horizontal axis. I mean, if we have to make the `batches` sit outsides the max-x, then might as well as align it with the numbers (10, 20, 30) on the horizontal axis. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113841374 Agreed, I'll try to update the PR tomorrow. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113690421 Can you provide the screenshot of the full UI so that we get sense of the overall aesthetics? Also @zsxwing what do you think? --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113694437 Looks great. In any case it shouldn't overlap with the bars. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-113474928 As promised, here are the screenshots: Before:  After:  --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-112905346 @zsxwing Can you have a look, please? --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-112913681 Merged build triggered. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-112913712 Merged build started. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-112914027 Hi @andrewor14, As I said on the JIRA, I'm not able to post screenshots at this time, sorry, but I might have the time this weekend. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-112914311 [Test build #35052 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35052/consoleFull) for PR 6845 at commit [`edd0936`](https://github.com/apache/spark/commit/edd093623f01e69cbef00b24b67809afea5ce49d). --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-112913463 Hi @BenFradet, could you post a screenshot of what this looks like before and after your change? --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-112913386 add to whitelist --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-112946384 [Test build #35052 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35052/console) for PR 6845 at commit [`edd0936`](https://github.com/apache/spark/commit/edd093623f01e69cbef00b24b67809afea5ce49d). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-112946470 Merged build finished. Test PASSed. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6845#issuecomment-112498868 Can one of the admins verify this patch? --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-8399][Streaming][Web UI] Overlap betwee...
GitHub user BenFradet opened a pull request: https://github.com/apache/spark/pull/6845 [SPARK-8399][Streaming][Web UI] Overlap between histograms and axis' name in Spark Streaming UI Moved where the X axis' name (#batches) is written in histograms in the spark streaming web ui so the histograms and the axis' name do not overlap. You can merge this pull request into a Git repository by running: $ git pull https://github.com/BenFradet/spark SPARK-8399 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/6845.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #6845 commit edd093623f01e69cbef00b24b67809afea5ce49d Author: BenFradet benjamin.fra...@gmail.com Date: 2015-06-16T16:58:53Z moved where the X axis' name (#batches) is written in histograms in the spark streaming web ui --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
