Github user holdenk commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-143591367
@srowen just pinging since its been a few days and no extra comments have
come in :)
---
If your project is set up for it, you can reply to this email and have your
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8879
---
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 srowen commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-143591695
Fair enough yeah I'm back online merging things now. Time for it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-143591763
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
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-143406654
Yeah looks good to me. Let me hold it open a day or so to see anyone has
comments
---
If your project is set up for it, you can reply to this email and have your
reply
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-143354947
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-143354966
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
Github user holdenk commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-143356109
@srowen so I refactored the shared code, but for serialization reasons I
put it in the companion object. Let me know if this looks good to you :)
---
If your project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-143380517
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-143379494
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-143381218
[Test build #43037 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43037/consoleFull)
for PR 8879 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-143391253
[Test build #43037 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43037/console)
for PR 8879 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-143391291
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/8879#issuecomment-143391289
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 holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/8879#discussion_r40404014
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala ---
@@ -531,6 +531,32 @@ class SQLContext(@transient val sparkContext:
SparkContext)
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/8879#discussion_r40403357
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala ---
@@ -531,6 +531,32 @@ class SQLContext(@transient val sparkContext:
SparkContext)
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/8879#discussion_r40295680
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala ---
@@ -531,6 +531,32 @@ class SQLContext(@transient val sparkContext:
SparkContext)
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/8879#discussion_r40388266
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala ---
@@ -531,6 +531,32 @@ class SQLContext(@transient val sparkContext:
SparkContext)
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-142760052
[Test build #42937 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42937/consoleFull)
for PR 8879 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-142762986
[Test build #42937 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42937/console)
for PR 8879 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-142764524
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-142759658
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-142763018
Test FAILed.
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/8879#issuecomment-142763017
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-142759670
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
Github user holdenk commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-142764390
I think thats spurious, 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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-142764540
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-142765051
[Test build #42943 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42943/consoleFull)
for PR 8879 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-142780660
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/8879#issuecomment-142780576
[Test build #42943 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42943/console)
for PR 8879 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8879#issuecomment-142780658
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
31 matches
Mail list logo