Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13264
---
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 user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/13264#issuecomment-221646004
LGTM. Merging to master / 2.0. Thanks!
---
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/13264#issuecomment-221564246
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13264#issuecomment-221563986
**[Test build #59274 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59274/consoleFull)**
for PR 13264 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13264#issuecomment-221564250
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13264#issuecomment-221539959
**[Test build #59274 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59274/consoleFull)**
for PR 13264 at commit
Github user lgieron commented on a diff in the pull request:
https://github.com/apache/spark/pull/13264#discussion_r64552539
--- Diff:
core/src/test/scala/org/apache/spark/ui/storage/StorageTabSuite.scala ---
@@ -179,6 +179,25 @@ class StorageTabSuite extends SparkFunSuite with
Github user lgieron commented on a diff in the pull request:
https://github.com/apache/spark/pull/13264#discussion_r64545642
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/StorageTab.scala
---
@@ -59,7 +59,15 @@ class StorageListener(storageStatusListener:
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13264#discussion_r64485134
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/StorageTab.scala
---
@@ -59,7 +59,15 @@ class StorageListener(storageStatusListener:
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/13264#discussion_r64459605
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/StorageTab.scala
---
@@ -59,7 +59,15 @@ class StorageListener(storageStatusListener:
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13264#discussion_r64453178
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/StorageTab.scala
---
@@ -59,7 +59,15 @@ class StorageListener(storageStatusListener:
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/13264#discussion_r64451727
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/StorageTab.scala
---
@@ -59,7 +59,15 @@ class StorageListener(storageStatusListener:
Github user lgieron commented on a diff in the pull request:
https://github.com/apache/spark/pull/13264#discussion_r64448718
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/StorageTab.scala
---
@@ -59,7 +59,15 @@ class StorageListener(storageStatusListener:
Github user lgieron commented on a diff in the pull request:
https://github.com/apache/spark/pull/13264#discussion_r64446719
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/StorageTab.scala
---
@@ -59,7 +59,15 @@ class StorageListener(storageStatusListener:
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13264#discussion_r6496
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/StorageTab.scala
---
@@ -59,7 +59,15 @@ class StorageListener(storageStatusListener:
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/13264#discussion_r64436937
--- Diff:
core/src/test/scala/org/apache/spark/ui/storage/StorageTabSuite.scala ---
@@ -179,6 +179,25 @@ class StorageTabSuite extends SparkFunSuite with
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/13264#discussion_r64436746
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/StorageTab.scala
---
@@ -59,7 +59,15 @@ class StorageListener(storageStatusListener:
Github user lgieron commented on a diff in the pull request:
https://github.com/apache/spark/pull/13264#discussion_r64365075
--- Diff:
core/src/test/scala/org/apache/spark/ui/storage/StorageTabSuite.scala ---
@@ -179,6 +179,25 @@ class StorageTabSuite extends SparkFunSuite with
Github user lgieron commented on a diff in the pull request:
https://github.com/apache/spark/pull/13264#discussion_r64364913
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/StorageTab.scala
---
@@ -59,7 +59,15 @@ class StorageListener(storageStatusListener:
Github user lgieron commented on a diff in the pull request:
https://github.com/apache/spark/pull/13264#discussion_r64354687
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/StorageTab.scala
---
@@ -59,7 +59,15 @@ class StorageListener(storageStatusListener:
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13264#discussion_r64315046
--- Diff:
core/src/test/scala/org/apache/spark/ui/storage/StorageTabSuite.scala ---
@@ -179,6 +179,25 @@ class StorageTabSuite extends SparkFunSuite with
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13264#discussion_r64315021
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/StorageTab.scala
---
@@ -59,7 +59,15 @@ class StorageListener(storageStatusListener:
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/13264#discussion_r64314987
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/StorageTab.scala
---
@@ -59,7 +59,15 @@ class StorageListener(storageStatusListener:
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/13264#discussion_r64312020
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/StorageTab.scala
---
@@ -59,7 +59,15 @@ class StorageListener(storageStatusListener:
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13264#issuecomment-221119529
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13264#issuecomment-221119527
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13264#issuecomment-221119370
**[Test build #59150 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59150/consoleFull)**
for PR 13264 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13264#issuecomment-221061309
**[Test build #59150 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59150/consoleFull)**
for PR 13264 at commit
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/13264#issuecomment-221059879
ok to test
---
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/13264#issuecomment-221046486
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
GitHub user lgieron opened a pull request:
https://github.com/apache/spark/pull/13264
SPARK-9044 Fix "Storage" tab in UI so that it reflects RDD name change.
## What changes were proposed in this pull request?
1. Making 'name' field of RDDInfo mutable.
2. In
31 matches
Mail list logo