Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11083#discussion_r53563531
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange.scala ---
@@ -261,10 +266,71 @@ object Exchange {
}
/**
+ * A [[Br
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11284#issuecomment-186777256
Thanks - I've merged this in 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
Github user huaxingao commented on a diff in the pull request:
https://github.com/apache/spark/pull/11250#discussion_r53563513
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/dstream/FileInputDStream.scala
---
@@ -163,8 +166,11 @@ class FileInputDStream[K, V, F <: Ne
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11083#discussion_r53563517
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange.scala ---
@@ -261,10 +266,71 @@ object Exchange {
}
/**
+ * A [[Br
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11083#discussion_r53563485
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/physical/partitioning.scala
---
@@ -351,3 +376,21 @@ case class PartitioningCollectio
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11083#discussion_r53563479
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/physical/partitioning.scala
---
@@ -76,6 +77,27 @@ case class OrderedDistribution(ord
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11083#discussion_r53563471
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/physical/partitioning.scala
---
@@ -213,7 +235,10 @@ case class RoundRobinPartitionin
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11284#issuecomment-186775975
**[Test build #2557 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2557/consoleFull)**
for PR 11284 at commit
[`d42d97d`](https://
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10757#issuecomment-186775925
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/10757#issuecomment-186775926
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/11288#issuecomment-186775746
**[Test build #51618 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51618/consoleFull)**
for PR 11288 at commit
[`79314b5`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11287#issuecomment-186775743
**[Test build #51616 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51616/consoleFull)**
for PR 11287 at commit
[`f2e6a25`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11285#issuecomment-186775782
**[Test build #51611 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51611/consoleFull)**
for PR 11285 at commit
[`c9bd987`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11286#issuecomment-186775751
**[Test build #51613 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51613/consoleFull)**
for PR 11286 at commit
[`3f2980a`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11142#issuecomment-186775641
**[Test build #51612 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51612/consoleFull)**
for PR 11142 at commit
[`ff9900b`](https://gi
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11083#discussion_r53563435
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/physical/partitioning.scala
---
@@ -76,6 +77,27 @@ case class OrderedDistribution(ord
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10757#issuecomment-186775478
**[Test build #51619 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51619/consoleFull)**
for PR 10757 at commit
[`e21fded`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11285#issuecomment-186775454
**[Test build #51614 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51614/consoleFull)**
for PR 11285 at commit
[`de512f6`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7#issuecomment-186775386
**[Test build #51617 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51617/consoleFull)**
for PR 7 at commit
[`4783c4c`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7#issuecomment-186775387
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7#issuecomment-186775388
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/7#issuecomment-186775311
**[Test build #51617 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51617/consoleFull)**
for PR 7 at commit
[`4783c4c`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11279#issuecomment-186775275
**[Test build #2556 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2556/consoleFull)**
for PR 11279 at commit
[`9f393f0`](https://
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11284#issuecomment-186775224
**[Test build #2557 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2557/consoleFull)**
for PR 11284 at commit
[`d42d97d`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11279#issuecomment-186775231
**[Test build #2556 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2556/consoleFull)**
for PR 11279 at commit
[`9f393f0`](https://g
Github user Earthson commented on the pull request:
https://github.com/apache/spark/pull/11237#issuecomment-186774933
@srowen Is this ok to 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 does not h
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/7#issuecomment-186774385
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 your project does not have this
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/11288#issuecomment-186773764
cc @davies
---
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
enabl
GitHub user rxin opened a pull request:
https://github.com/apache/spark/pull/11288
[SPARK-13420][SQL] Rename Subquery logical plan to SubqueryAlias
## What changes were proposed in this pull request?
This patch renames logical.Subquery to logical.SubqueryAlias, which is a
more a
401 - 429 of 429 matches
Mail list logo