Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/7692
---
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 enab
Github user sarutak commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-127599315
O.K. Merging into `master` and `branch-1.5`. Thanks @carsonwang !
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-127597239
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 a
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/7692#discussion_r36184341
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala ---
@@ -1218,7 +1216,7 @@ private[ui] class TaskPagedTable(
Seq(("Errors",
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-127476144
[Test build #39636 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39636/console)
for PR 7692 at commit
[`03c7168`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-127476336
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
Github user sarutak commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-127469861
Works well in my machine.
---
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 t
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-127455676
[Test build #39636 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39636/consoleFull)
for PR 7692 at commit
[`03c7168`](https://gith
Github user carsonwang commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-127455197
Thanks @sarutak . The style issues are fixed.
---
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-127454719
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-127454684
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 h
Github user sarutak commented on a diff in the pull request:
https://github.com/apache/spark/pull/7692#discussion_r36143508
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/RDDPage.scala ---
@@ -120,23 +151,159 @@ private[ui] class RDDPage(parent: StorageTab)
extends WebUI
Github user sarutak commented on a diff in the pull request:
https://github.com/apache/spark/pull/7692#discussion_r36143409
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/RDDPage.scala ---
@@ -44,8 +57,34 @@ private[ui] class RDDPage(parent: StorageTab) extends
WebUIPage
Github user sarutak commented on a diff in the pull request:
https://github.com/apache/spark/pull/7692#discussion_r36143267
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/RDDPage.scala ---
@@ -32,6 +33,18 @@ private[ui] class RDDPage(parent: StorageTab) extends
WebUIPage
Github user sarutak commented on a diff in the pull request:
https://github.com/apache/spark/pull/7692#discussion_r36143160
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/RDDPage.scala ---
@@ -44,8 +57,34 @@ private[ui] class RDDPage(parent: StorageTab) extends
WebUIPage
Github user sarutak commented on a diff in the pull request:
https://github.com/apache/spark/pull/7692#discussion_r36143016
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/RDDPage.scala ---
@@ -120,23 +151,159 @@ private[ui] class RDDPage(parent: StorageTab)
extends WebUI
Github user sarutak commented on a diff in the pull request:
https://github.com/apache/spark/pull/7692#discussion_r36142274
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/RDDPage.scala ---
@@ -17,12 +17,13 @@
package org.apache.spark.ui.storage
+impor
Github user sarutak commented on a diff in the pull request:
https://github.com/apache/spark/pull/7692#discussion_r36142297
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/RDDPage.scala ---
@@ -17,12 +17,13 @@
package org.apache.spark.ui.storage
+impor
Github user sarutak commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-127428355
O.K, I'll take a look 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 project does not have
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-127364894
Bump. @zsxwing (or maybe @sarutak if you have cycles), let's see about
pulling this in for 1.5?
---
If your project is set up for it, you can reply to this email and
Github user carsonwang commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125860669
Screenshots attached.
---
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 th
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125708496
@carsonwang would you mind posting a screenshot of what this looks like
before and after?
---
If your project is set up for it, you can reply to this email and have
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125708186
@zsxwing could you review 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 n
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125508066
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125507942
[Test build #38671 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/38671/console)
for PR 7692 at commit
[`612c18c`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125500025
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125499822
[Test build #131 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/131/console)
for PR 7692 at commit
[`612c18c`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125466750
[Test build #131 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/131/consoleFull)
for PR 7692 at commit
[`612c18c`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125466892
[Test build #38671 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/38671/consoleFull)
for PR 7692 at commit
[`612c18c`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125466707
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125466688
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 h
Github user carsonwang commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125466146
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125466225
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125466209
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125399910
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125399901
[Test build #126 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/126/console)
for PR 7692 at commit
[`612c18c`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125399267
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125398006
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125398089
[Test build #126 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/126/consoleFull)
for PR 7692 at commit
[`612c18c`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125397994
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125397672
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125397678
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
Github user carsonwang commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125397558
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125196657
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125196573
**[Test build #38531 timed
out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/38531/console)**
for PR 7692 at commit
[`612c18c`](https://github
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125152956
[Test build #38531 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/38531/consoleFull)
for PR 7692 at commit
[`612c18c`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125151619
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7692#issuecomment-125127459
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 h
GitHub user carsonwang opened a pull request:
https://github.com/apache/spark/pull/7692
[SPARK-2016][WebUI]RDD partition table pagination for the RDD Page
Add pagination for the RDD page to avoid unresponsive UI when the number of
the RDD partitions is large.
You can merge this pu
49 matches
Mail list logo