Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-169413921
Thanks! Merging 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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9858
---
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 saucam commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-169523800
Thanks for the comments and the merge :)
---
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/9858#issuecomment-168932871
**[Test build #48751 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48751/consoleFull)**
for PR 9858 at commit
Github user saucam commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-168934153
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 project does not have this feature
enabled
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-168952176
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/9858#issuecomment-168952173
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-168951738
**[Test build #48751 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48751/consoleFull)**
for PR 9858 at commit
Github user saucam commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-168956141
Seems some other issue , tests in pyspark mllib failing ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/9858#discussion_r48874115
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -361,8 +361,15 @@ private[sql] abstract class SparkStrategies
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/9858#discussion_r48873698
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -361,8 +361,15 @@ private[sql] abstract class SparkStrategies
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-169241986
**[Test build #48837 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48837/consoleFull)**
for PR 9858 at commit
Github user saucam commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-169240395
Hey @marmbrus have reverted to 46e7419
---
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/9858#issuecomment-169258458
**[Test build #48837 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48837/consoleFull)**
for PR 9858 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-169258612
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-169258614
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/9858#discussion_r48777897
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -361,8 +361,15 @@ private[sql] abstract class SparkStrategies
Github user saucam commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-167422237
Hey @marmbrus sorry for the delay in this update, I have added the same
thing to the planner. Also rebased to latest master. How does it look now ?
---
If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-167423222
**[Test build #48353 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48353/consoleFull)**
for PR 9858 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-167432879
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/9858#issuecomment-167432877
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/9858#issuecomment-167426083
**[Test build #48354 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48354/consoleFull)**
for PR 9858 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-167423751
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-167423743
**[Test build #48353 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48353/consoleFull)**
for PR 9858 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-167423754
Test FAILed.
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/9858#issuecomment-167432837
**[Test build #48354 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48354/consoleFull)**
for PR 9858 at commit
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/9858#discussion_r46209620
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -488,6 +488,12 @@ private[sql] case class
Github user saucam commented on a diff in the pull request:
https://github.com/apache/spark/pull/9858#discussion_r46020227
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -488,6 +488,12 @@ private[sql] case class
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/9858#discussion_r46002572
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -488,6 +488,12 @@ private[sql] case class
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-159799524
**[Test build #46742 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46742/consoleFull)**
for PR 9858 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-159814177
**[Test build #46742 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46742/consoleFull)**
for PR 9858 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-159814439
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/9858#issuecomment-159814436
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-158349971
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/9858#issuecomment-158349801
**[Test build #46404 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46404/consoleFull)**
for PR 9858 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-158349969
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 marmbrus commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-158517844
I think it would be more general to do this during exchange planning.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-158318742
**[Test build #46404 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46404/consoleFull)**
for PR 9858 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-158314376
**[Test build #46403 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46403/consoleFull)**
for PR 9858 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9858#issuecomment-158314388
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/9858#issuecomment-158314394
Test FAILed.
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/9858#issuecomment-158314030
**[Test build #46403 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46403/consoleFull)**
for PR 9858 at commit
42 matches
Mail list logo