Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11580
---
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 ena
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11580#issuecomment-196214875
Merged to master
---
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 featu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11580#issuecomment-196051051
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11580#issuecomment-196051048
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11580#issuecomment-196050740
**[Test build #53035 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53035/consoleFull)**
for PR 11580 at commit
[`bb8b83a`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11580#issuecomment-196016180
**[Test build #53035 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53035/consoleFull)**
for PR 11580 at commit
[`bb8b83a`](https://gi
Github user wilson8 commented on a diff in the pull request:
https://github.com/apache/spark/pull/11580#discussion_r55941313
--- Diff: core/src/test/scala/org/apache/spark/ContextCleanerSuite.scala ---
@@ -578,18 +578,27 @@ class CleanerTester(
}
private
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11580#discussion_r55935805
--- Diff: core/src/test/scala/org/apache/spark/ContextCleanerSuite.scala ---
@@ -578,18 +578,27 @@ class CleanerTester(
}
private def uncle
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11580#discussion_r55935798
--- Diff: core/src/test/scala/org/apache/spark/ContextCleanerSuite.scala ---
@@ -442,25 +442,25 @@ class CleanerTester(
checkpointIds: Seq[Long] = Seq
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11580#issuecomment-195931183
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11580#issuecomment-195931184
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11580#issuecomment-195931137
**[Test build #53024 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53024/consoleFull)**
for PR 11580 at commit
[`c1e4e41`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11580#issuecomment-195911950
**[Test build #53024 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53024/consoleFull)**
for PR 11580 at commit
[`c1e4e41`](https://gi
Github user wilson8 commented on a diff in the pull request:
https://github.com/apache/spark/pull/11580#discussion_r55933247
--- Diff: core/src/test/scala/org/apache/spark/ContextCleanerSuite.scala ---
@@ -579,17 +579,20 @@ class CleanerTester(
private def uncle
Github user wilson8 commented on a diff in the pull request:
https://github.com/apache/spark/pull/11580#discussion_r55933241
--- Diff: core/src/test/scala/org/apache/spark/ContextCleanerSuite.scala ---
@@ -442,25 +442,25 @@ class CleanerTester(
checkpointIds: Seq[Lo
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11580#issuecomment-195657855
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11580#issuecomment-195657858
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11580#issuecomment-195657820
**[Test build #52968 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52968/consoleFull)**
for PR 11580 at commit
[`6721259`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11580#issuecomment-195611810
**[Test build #52968 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52968/consoleFull)**
for PR 11580 at commit
[`6721259`](https://gi
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/11580#issuecomment-195611297
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 feature
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11580#issuecomment-194709904
This needs a rebase anyway so I left a few more comments. Otherwise LGTM
pending tests
---
If your project is set up for it, you can reply to this email and have your
r
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11580#discussion_r55641453
--- Diff: core/src/test/scala/org/apache/spark/ContextCleanerSuite.scala ---
@@ -579,17 +579,20 @@ class CleanerTester(
private def uncleanedResou
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11580#discussion_r55641420
--- Diff: core/src/test/scala/org/apache/spark/ContextCleanerSuite.scala ---
@@ -442,25 +442,25 @@ class CleanerTester(
checkpointIds: Seq[Long] = Seq
Github user wilson8 commented on a diff in the pull request:
https://github.com/apache/spark/pull/11580#discussion_r55634450
--- Diff: core/src/test/scala/org/apache/spark/ContextCleanerSuite.scala ---
@@ -586,10 +586,12 @@ class CleanerTester(
}
private
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/11580#discussion_r55400287
--- Diff: core/src/test/scala/org/apache/spark/ContextCleanerSuite.scala ---
@@ -586,10 +586,12 @@ class CleanerTester(
}
private def isAll
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11580#issuecomment-193893662
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 p
GitHub user wilson8 opened a pull request:
https://github.com/apache/spark/pull/11580
[SPARK-13746][Tests]stop using deprecated SynchronizedSet
trait SynchronizedSet in package mutable is deprecated
You can merge this pull request into a Git repository by running:
27 matches
Mail list logo