[GitHub] spark pull request: [SPARK-13618][STREAMING][WEB-UI][WIP] Make Str...

2016-03-02 Thread proflin
Github user proflin commented on the pull request: https://github.com/apache/spark/pull/11470#issuecomment-191261292 2 things I'm still working on: - better deal with situations where `numRecordsLimit`s are mixed with `Some(...)` from some batches and 'None' from th

[GitHub] spark pull request: [SPARK-13618][STREAMING][WEB-UI][WIP] Make Str...

2016-03-02 Thread proflin
GitHub user proflin opened a pull request: https://github.com/apache/spark/pull/11470 [SPARK-13618][STREAMING][WEB-UI][WIP] Make Streaming web UI page display rate-limit lines on statistics graph ## What changes were proposed in this pull request? This PR makes Streaming

[GitHub] spark pull request: [Minor][Streaming] Fix a minor naming issue in...

2016-02-26 Thread proflin
Github user proflin commented on the pull request: https://github.com/apache/spark/pull/11385#issuecomment-189159311 Thank you all for reviewing this. --- 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

[GitHub] spark pull request: [Minor][Streaming] Fix a minor naming issue in...

2016-02-25 Thread proflin
Github user proflin commented on a diff in the pull request: https://github.com/apache/spark/pull/11385#discussion_r54211371 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/api/java/JavaDStreamLike.scala --- @@ -50,7 +50,7 @@ trait JavaDStreamLike[T, This

[GitHub] spark pull request: [Minor][Streaming] Fix a minor naming issue in...

2016-02-25 Thread proflin
Github user proflin commented on the pull request: https://github.com/apache/spark/pull/11385#issuecomment-189128154 This PR fixes a minor naming issue first introduced in d182a57cae6455804773db23d9498d2dcdd02172. @zsxwing would you mind taking a look at this? Thanks. --- If

[GitHub] spark pull request: [Minor][Streaming] Fix a minor naming issue in...

2016-02-25 Thread proflin
GitHub user proflin opened a pull request: https://github.com/apache/spark/pull/11385 [Minor][Streaming] Fix a minor naming issue in JavaDStreamLike You can merge this pull request into a Git repository by running: $ git pull https://github.com/proflin/spark Fix-minor-naming

[GitHub] spark pull request: [SPARK-13468][Web UI] Fix a corner case where ...

2016-02-25 Thread proflin
Github user proflin commented on the pull request: https://github.com/apache/spark/pull/11368#issuecomment-188696112 @andrewor14 @zsxwing would you mind taking a look when you have time? Thanks! --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-13468][Web UI] Fix a corner case where ...

2016-02-25 Thread proflin
GitHub user proflin opened a pull request: https://github.com/apache/spark/pull/11368 [SPARK-13468][Web UI] Fix a corner case where the page UI should show DAG but it doesn't show When uses clicks more than one time on any stage in the DAG graph on the *Job* web UI page, man

[GitHub] spark pull request: [SQL][MINOR] Fix one little mismatched comment...

2016-01-19 Thread proflin
Github user proflin commented on the pull request: https://github.com/apache/spark/pull/10824#issuecomment-172781409 @rxin Thanks for the prompt reviewing & merging. :-) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] spark pull request: [SQL][MINOR] Fix one little mismatched comment...

2016-01-18 Thread proflin
GitHub user proflin opened a pull request: https://github.com/apache/spark/pull/10824 [SQL][MINOR] Fix one little mismatched comment according to the codes in interface.scala You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] spark pull request: [SPARK-12304][STREAMING] Make Spark Streaming ...

2015-12-15 Thread proflin
Github user proflin commented on the pull request: https://github.com/apache/spark/pull/10318#issuecomment-165012493 @zsxwing Thanks for reviewing and merging so soon. :-) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] spark pull request: [SPARK-12304][STREAMING] Make Spark Streaming ...

2015-12-15 Thread proflin
Github user proflin commented on a diff in the pull request: https://github.com/apache/spark/pull/10318#discussion_r47722046 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/ui/StreamingPage.scala --- @@ -392,9 +392,14 @@ private[ui] class StreamingPage(parent

[GitHub] spark pull request: [SPARK-12304][STREAMING] Make Spark Streaming ...

2015-12-15 Thread proflin
Github user proflin commented on the pull request: https://github.com/apache/spark/pull/10318#issuecomment-164938969 @zsxwing could you take a look at this again, please? :-) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark pull request: [SPARK-12304][STREAMING] Make Spark Streaming ...

2015-12-15 Thread proflin
GitHub user proflin opened a pull request: https://github.com/apache/spark/pull/10318 [SPARK-12304][STREAMING] Make Spark Streaming web UI display more fri… …endly Receiver graphs Currently, the Spark Streaming web UI uses the same maxY when displays 'Input Rate

[GitHub] spark pull request: [SPARK-12304][STREAMING] Make Spark Streaming ...

2015-12-14 Thread proflin
Github user proflin commented on the pull request: https://github.com/apache/spark/pull/10276#issuecomment-164630203 Closing this PR. I'll start a new PR readdressing this issue as well as resolving the conflicts. Thanks again for the reviews. --- If your project i

[GitHub] spark pull request: [SPARK-12304][STREAMING] Make Spark Streaming ...

2015-12-14 Thread proflin
Github user proflin closed the pull request at: https://github.com/apache/spark/pull/10276 --- 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

[GitHub] spark pull request: [SPARK-12304][STREAMING] Make Spark Streaming ...

2015-12-14 Thread proflin
Github user proflin commented on a diff in the pull request: https://github.com/apache/spark/pull/10276#discussion_r47591823 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/ui/StreamingPage.scala --- @@ -392,8 +392,12 @@ private[ui] class StreamingPage(parent

[GitHub] spark pull request: [SPARK-12304][STREAMING] Make Spark Streaming ...

2015-12-14 Thread proflin
Github user proflin commented on a diff in the pull request: https://github.com/apache/spark/pull/10276#discussion_r47591698 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/ui/StreamingPage.scala --- @@ -392,8 +392,12 @@ private[ui] class StreamingPage(parent

[GitHub] spark pull request: [SPARK-12304][STREAMING] Make Spark Streaming ...

2015-12-12 Thread proflin
Github user proflin commented on the pull request: https://github.com/apache/spark/pull/10276#issuecomment-164136085 @zsxwing could you take a look at 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

[GitHub] spark pull request: [SPARK-12304][STREAMING] Make Spark Streaming ...

2015-12-12 Thread proflin
GitHub user proflin opened a pull request: https://github.com/apache/spark/pull/10276 [SPARK-12304][STREAMING] Make Spark Streaming web UI display more fri… …endly Receiver graphs Currently, the Spark Streaming web UI uses the same maxY when displays 'Input Rate

[GitHub] spark pull request: [SPARK-12273][STREAMING] Make Spark Streaming ...

2015-12-11 Thread proflin
Github user proflin commented on the pull request: https://github.com/apache/spark/pull/10264#issuecomment-164087083 Thanks @zsxwing for reviewing so soon :-) --- 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

[GitHub] spark pull request: [SPARK-12273][STREAMING] Make Spark Streaming ...

2015-12-10 Thread proflin
Github user proflin commented on the pull request: https://github.com/apache/spark/pull/10264#issuecomment-163845082 /cc @tdas @zsxwing --- 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

[GitHub] spark pull request: [SPARK-12273][STREAMING] Make Spark Streaming ...

2015-12-10 Thread proflin
GitHub user proflin opened a pull request: https://github.com/apache/spark/pull/10264 [SPARK-12273][STREAMING] Make Spark Streaming web UI list Receivers in order Currently the Streaming web UI does NOT list Receivers in order; however, it seems more convenient for the users if