Github user adrian-wang commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-112260579
@justmytwospence yes.
---
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 t
Github user justmytwospence commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-112235776
Is this feature limited to equi-joins?
---
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 pr
Github user adrian-wang commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93613197
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 not have this feature
ena
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93569492
I manually fixed the conflicts while merging to master. Thanks! I'm
excited to test out the performance of this new feature :)
---
If your project is set up for it, y
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/5208
---
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 enab
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93257614
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93257603
[Test build #30321 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30321/consoleFull)
for PR 5208 at commit
[`2493b9f`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93224019
[Test build #30304 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30304/consoleFull)
for PR 5208 at commit
[`ec8061b`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93224051
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93220011
[Test build #30321 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30321/consoleFull)
for PR 5208 at commit
[`2493b9f`](https://githu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93219531
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93219524
[Test build #30319 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30319/consoleFull)
for PR 5208 at commit
[`5049d88`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93219080
[Test build #30319 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30319/consoleFull)
for PR 5208 at commit
[`5049d88`](https://githu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93218845
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93218835
[Test build #30315 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30315/consoleFull)
for PR 5208 at commit
[`f91a2ae`](https://gith
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28396290
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -157,28 +205,61 @@ private[sql] case class AddExchange(sqlContext:
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28396071
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -87,7 +126,12 @@ case class Exchange(newPartitioning: Partitioning,
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28395998
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -87,7 +126,12 @@ case class Exchange(newPartitioning: Partitioning,
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28395922
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -157,28 +205,61 @@ private[sql] case class AddExchange(sqlContext:
SQ
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28395812
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -157,28 +205,61 @@ private[sql] case class AddExchange(sqlContext:
SQLCo
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93215769
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93215758
[Test build #30309 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30309/consoleFull)
for PR 5208 at commit
[`f515cd2`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93213692
[Test build #30315 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30315/consoleFull)
for PR 5208 at commit
[`f91a2ae`](https://githu
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28395453
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -157,28 +205,61 @@ private[sql] case class AddExchange(sqlContext:
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28395272
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/basicOperators.scala ---
@@ -102,6 +106,8 @@ case class Limit(limit: Int, child: SparkPlan)
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28395119
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -157,28 +205,61 @@ private[sql] case class AddExchange(sqlContext:
SQLCo
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93205970
[Test build #30309 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30309/consoleFull)
for PR 5208 at commit
[`f515cd2`](https://githu
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28394994
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -157,28 +205,61 @@ private[sql] case class AddExchange(sqlContext:
SQLCo
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28394950
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -157,28 +205,61 @@ private[sql] case class AddExchange(sqlContext:
SQLCo
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28394813
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -19,24 +19,39 @@ package org.apache.spark.sql.execution
import
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28394599
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -120,27 +161,34 @@ case class Exchange(newPartitioning: Partitioning,
ch
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28394315
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/JoinSuite.scala ---
@@ -91,6 +94,16 @@ class JoinSuite extends QueryTest with
BeforeAndAfterEach {
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28394224
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/JoinSuite.scala ---
@@ -51,6 +51,8 @@ class JoinSuite extends QueryTest with BeforeAndAfterEach
{
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93194500
[Test build #30304 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30304/consoleFull)
for PR 5208 at commit
[`ec8061b`](https://githu
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93190389
@yhuai can you do another pass over `Exchange.scala`? I made several
changes.
---
If your project is set up for it, you can reply to this email and have your
reply app
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93189576
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93189565
[Test build #30299 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30299/consoleFull)
for PR 5208 at commit
[`413fd24`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93173575
[Test build #30299 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30299/consoleFull)
for PR 5208 at commit
[`413fd24`](https://githu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93164496
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93164474
[Test build #30289 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30289/consoleFull)
for PR 5208 at commit
[`b198278`](https://gith
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93157431
https://github.com/adrian-wang/spark/pull/3
---
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 proje
Github user adrian-wang commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93136064
Thanks! just in case your PR will conflict, please note that I just update
my PR.
---
If your project is set up for it, you can reply to this email and have your
rep
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93134866
[Test build #30289 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30289/consoleFull)
for PR 5208 at commit
[`b198278`](https://githu
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-93130870
BTW, as I discussed with @chenghao-intel today, I think this is very close
to merging. There are a few things I'd like to clean up, but I propose I just
make a PR again
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28386536
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/basicOperators.scala ---
@@ -41,6 +41,15 @@ case class Project(projectList: Seq[NamedExpr
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28350524
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/basicOperators.scala ---
@@ -41,6 +41,15 @@ case class Project(projectList: Seq[NamedExpress
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92743653
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92743572
[Test build #30237 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30237/consoleFull)
for PR 5208 at commit
[`c8e82a3`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92696369
[Test build #30237 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30237/consoleFull)
for PR 5208 at commit
[`c8e82a3`](https://githu
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92693283
[Test build #30235 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30235/consoleFull)
for PR 5208 at commit
[`6e897dd`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92693289
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92692706
[Test build #30235 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30235/consoleFull)
for PR 5208 at commit
[`6e897dd`](https://githu
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28307147
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeJoin.scala
---
@@ -0,0 +1,172 @@
+/*
+ * Licensed to the Apache S
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28302364
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/basicOperators.scala ---
@@ -41,6 +41,15 @@ case class Project(projectList: Seq[NamedExpr
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28302328
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala ---
@@ -72,6 +72,12 @@ abstract class SparkPlan extends QueryPlan[SparkPl
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28302153
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -168,14 +190,16 @@ private[sql] case class AddExchange(sqlContext:
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28302133
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -69,7 +72,16 @@ case class Exchange(newPartitioning: Partitioning,
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92526609
I think this is getting pretty close! If you run into trouble implementing
some of my suggestions on the `SparkPlan` interfaces for ordering please let me
know and I ca
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28287371
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeJoin.scala
---
@@ -0,0 +1,172 @@
+/*
+ * Licensed to the Apache Soft
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28287227
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/basicOperators.scala ---
@@ -41,6 +41,15 @@ case class Project(projectList: Seq[NamedExpress
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28287180
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -33,7 +32,11 @@ import org.apache.spark.util.MutablePair
* :: De
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28287120
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -168,14 +190,16 @@ private[sql] case class AddExchange(sqlContext:
SQ
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28286721
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala ---
@@ -72,6 +72,12 @@ abstract class SparkPlan extends QueryPlan[SparkPlan]
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28286620
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -69,7 +72,16 @@ case class Exchange(newPartitioning: Partitioning,
ch
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92082262
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92082260
[Test build #30122 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30122/consoleFull)
for PR 5208 at commit
[`2875ef2`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92067742
[Test build #30122 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30122/consoleFull)
for PR 5208 at commit
[`2875ef2`](https://githu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92006103
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92006101
[Test build #30110 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30110/consoleFull)
for PR 5208 at commit
[`61d7f49`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92005972
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92005970
[Test build #30109 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30109/consoleFull)
for PR 5208 at commit
[`00a4430`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92005298
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92005296
[Test build #30107 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30107/consoleFull)
for PR 5208 at commit
[`078d69b`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92005254
[Test build #30110 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30110/consoleFull)
for PR 5208 at commit
[`61d7f49`](https://githu
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92005142
[Test build #30109 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30109/consoleFull)
for PR 5208 at commit
[`00a4430`](https://githu
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-92003179
[Test build #30107 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30107/consoleFull)
for PR 5208 at commit
[`078d69b`](https://githu
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28200329
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Aggregate.scala ---
@@ -60,6 +60,8 @@ case class Aggregate(
override def out
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28200178
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeJoin.scala
---
@@ -0,0 +1,163 @@
+/*
+ * Licensed to the Apache Softwar
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28200168
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeJoin.scala
---
@@ -0,0 +1,163 @@
+/*
+ * Licensed to the Apache Softwar
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28200163
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeJoin.scala
---
@@ -0,0 +1,163 @@
+/*
+ * Licensed to the Apache Softwar
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28200167
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeJoin.scala
---
@@ -0,0 +1,163 @@
+/*
+ * Licensed to the Apache Softwar
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-91936211
Thanks a lot for working on this! I'm very excited to have this feature.
My biggest comment is that while I like the logical separation of `ordering`
from `partitionin
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28199028
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeJoin.scala
---
@@ -0,0 +1,163 @@
+/*
+ * Licensed to the Apache Soft
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28199002
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeJoin.scala
---
@@ -0,0 +1,163 @@
+/*
+ * Licensed to the Apache Soft
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28198969
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeJoin.scala
---
@@ -0,0 +1,163 @@
+/*
+ * Licensed to the Apache Soft
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28198952
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeJoin.scala
---
@@ -0,0 +1,163 @@
+/*
+ * Licensed to the Apache Soft
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28198938
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeJoin.scala
---
@@ -0,0 +1,163 @@
+/*
+ * Licensed to the Apache Soft
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28198929
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -162,6 +161,11 @@ private[sql] case class AddExchange(sqlContext:
SQL
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28198910
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeJoin.scala
---
@@ -0,0 +1,164 @@
+/*
+ * Licensed to the Apache Soft
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28198903
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeJoin.scala
---
@@ -0,0 +1,164 @@
+/*
+ * Licensed to the Apache Soft
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28198883
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala ---
@@ -72,6 +72,12 @@ abstract class SparkPlan extends QueryPlan[SparkPlan]
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28198853
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -90,6 +90,13 @@ private[sql] abstract class SparkStrategies ext
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28198847
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala ---
@@ -177,6 +183,7 @@ private[sql] trait LeafNode extends SparkPlan with
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28198836
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala ---
@@ -72,6 +72,12 @@ abstract class SparkPlan extends QueryPlan[SparkPlan]
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28198810
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLConf.scala ---
@@ -144,6 +145,12 @@ private[sql] class SQLConf extends Serializable {
getCo
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28198823
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Aggregate.scala ---
@@ -60,6 +60,8 @@ case class Aggregate(
override def output
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5208#discussion_r28198804
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLConf.scala ---
@@ -27,6 +27,7 @@ private[spark] object SQLConf {
val COLUMN_BATCH_SIZE = "spa
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-91868560
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-91868553
[Test build #30075 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30075/consoleFull)
for PR 5208 at commit
[`3af6ba5`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5208#issuecomment-91844058
[Test build #30075 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30075/consoleFull)
for PR 5208 at commit
[`3af6ba5`](https://githu
1 - 100 of 152 matches
Mail list logo