Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/5497
---
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 marmbrus commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-94608587
Thanks! 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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-94131617
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/5497#issuecomment-94131614
[Test build #30513 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30513/consoleFull)
for PR 5497 at commit
[`da562c5`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-94123413
[Test build #30513 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30513/consoleFull)
for PR 5497 at commit
[`da562c5`](https://githu
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5497#discussion_r28563376
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLConf.scala ---
@@ -139,6 +141,8 @@ private[sql] class SQLConf extends Serializable {
*/
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5497#discussion_r28563536
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLConf.scala ---
@@ -139,6 +141,8 @@ private[sql] class SQLConf extends Serializable {
*/
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5497#discussion_r28563467
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlSerializer2.scala
---
@@ -0,0 +1,411 @@
+/*
+ * Licensed to the Apache Soft
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5497#discussion_r28563390
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -77,9 +79,30 @@ case class Exchange(
}
}
-
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/5497#discussion_r28563401
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -77,9 +79,30 @@ case class Exchange(
}
}
-
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-93614975
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/5497#issuecomment-93614969
[Test build #30384 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30384/consoleFull)
for PR 5497 at commit
[`09e587a`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-93612773
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/5497#issuecomment-93612763
[Test build #30382 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30382/consoleFull)
for PR 5497 at commit
[`791b96a`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-93602324
[Test build #30384 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30384/consoleFull)
for PR 5497 at commit
[`09e587a`](https://githu
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-93600650
[Test build #30382 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30382/consoleFull)
for PR 5497 at commit
[`791b96a`](https://githu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-93163340
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/5497#issuecomment-93163336
[Test build #30288 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30288/consoleFull)
for PR 5497 at commit
[`3e09655`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-93127680
[Test build #30288 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30288/consoleFull)
for PR 5497 at commit
[`3e09655`](https://githu
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/5497#discussion_r28365182
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlSerializer2.scala
---
@@ -0,0 +1,408 @@
+/*
+ * Licensed to the Apache Softwar
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-93021985
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/5497#issuecomment-93021976
[Test build #30259 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30259/consoleFull)
for PR 5497 at commit
[`43b9fb4`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-93016164
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/5497#issuecomment-93016119
[Test build #30255 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30255/consoleFull)
for PR 5497 at commit
[`8297732`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-92991958
[Test build #30259 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30259/consoleFull)
for PR 5497 at commit
[`43b9fb4`](https://githu
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-92982984
[Test build #30255 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30255/consoleFull)
for PR 5497 at commit
[`8297732`](https://githu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-92593118
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/5497#issuecomment-92593096
[Test build #30215 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30215/consoleFull)
for PR 5497 at commit
[`c9373c8`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-92590399
[Test build #30215 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30215/consoleFull)
for PR 5497 at commit
[`c9373c8`](https://githu
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/5497#discussion_r28286461
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlSerializer2.scala
---
@@ -0,0 +1,378 @@
+/*
+ * Licensed to the Apache Softwar
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/5497#discussion_r28286134
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlSerializer2.scala
---
@@ -0,0 +1,378 @@
+/*
+ * Licensed to the Apache Softwar
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/5497#discussion_r28286063
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlSerializer2.scala
---
@@ -0,0 +1,378 @@
+/*
+ * Licensed to the Apache Softwar
Github user chenghao-intel commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-92514199
@yhuai this is a really cool improvement, definitely will improve the
performance a lot. I have some of the comments about the future improvement(of
course we can
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/5497#discussion_r28283974
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLConf.scala ---
@@ -139,6 +141,8 @@ private[sql] class SQLConf extends Serializable {
*
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/5497#discussion_r28283598
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlSerializer2.scala
---
@@ -0,0 +1,378 @@
+/*
+ * Licensed to the Apach
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/5497#discussion_r28283271
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Exchange.scala ---
@@ -45,6 +46,27 @@ case class Exchange(newPartitioning: Partitionin
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/5497#discussion_r28282909
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlSerializer2.scala
---
@@ -0,0 +1,378 @@
+/*
+ * Licensed to the Apach
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/5497#discussion_r28282279
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlSerializer2.scala
---
@@ -0,0 +1,378 @@
+/*
+ * Licensed to the Apach
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/5497#discussion_r28282630
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlSerializer2.scala
---
@@ -0,0 +1,378 @@
+/*
+ * Licensed to the Apach
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/5497#discussion_r28282545
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlSerializer2.scala
---
@@ -0,0 +1,378 @@
+/*
+ * Licensed to the Apach
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/5497#discussion_r28282234
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlSerializer2.scala
---
@@ -0,0 +1,378 @@
+/*
+ * Licensed to the Apach
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-92507937
[Test build #30196 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30196/consoleFull)
for PR 5497 at commit
[`2379eeb`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-92507955
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/5497#issuecomment-92502644
[Test build #30196 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30196/consoleFull)
for PR 5497 at commit
[`2379eeb`](https://githu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-92500617
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/5497#issuecomment-92500606
[Test build #30194 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30194/consoleFull)
for PR 5497 at commit
[`39704ab`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5497#issuecomment-92500556
[Test build #30194 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30194/consoleFull)
for PR 5497 at commit
[`39704ab`](https://githu
GitHub user yhuai opened a pull request:
https://github.com/apache/spark/pull/5497
[SPARK-6368][SQL] Build a specialized serializer for Exchange operator.
JIRA: https://issues.apache.org/jira/browse/SPARK-6368
You can merge this pull request into a Git repository by running:
$
48 matches
Mail list logo