Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/10844#discussion_r51068499
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicOperators.scala
---
@@ -88,6 +88,12 @@ case class Generate(
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/10844#discussion_r51068376
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/QueryPlan.scala
---
@@ -17,16 +17,31 @@
package
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/10844#discussion_r51068566
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicOperators.scala
---
@@ -146,6 +172,26 @@ case class Union(children:
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/10844#discussion_r51068802
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/QueryPlan.scala
---
@@ -17,16 +17,31 @@
package
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10844#issuecomment-174926239
**[Test build #50083 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50083/consoleFull)**
for PR 10844 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10844#issuecomment-17537
**[Test build #50161 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50161/consoleFull)**
for PR 10844 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10844#issuecomment-175375683
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/10844#issuecomment-175375684
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/10844#issuecomment-174883150
**[Test build #50083 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50083/consoleFull)**
for PR 10844 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10844#issuecomment-175354244
**[Test build #50161 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50161/consoleFull)**
for PR 10844 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10844#issuecomment-174926671
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/10844#issuecomment-174926676
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user sameeragarwal commented on the pull request:
https://github.com/apache/spark/pull/10844#issuecomment-173787321
@hvanhovell added, 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 does
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/10844#issuecomment-173796157
@sameeragarwal I just saw it. Thank you! I will do the code changes after
this is merged.
---
If your project is set up for it, you can reply to this email and
14 matches
Mail list logo