Github user gberger commented on the issue:
https://github.com/apache/spark/pull/19792
@ueshin
The reason that I modified the case for StructType is that, in
session.py#341, for each Pandas DF row we obtain a StructType with StructFields
mapping column names to value type; t
Github user gberger commented on the issue:
https://github.com/apache/spark/pull/19792
@ebuildy fixed
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19792
**[Test build #84105 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84105/testReport)**
for PR 19792 at commit
[`b29434e`](https://github.com/apache/spark/commit/b2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19786
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19786
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84098/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19786
**[Test build #84098 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84098/testReport)**
for PR 19786 at commit
[`8cfa0b2`](https://github.com/apache/spark/commit/8
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19756
can you also explain the cause of this bug instead of just describing the
bug? By looking at your code, seems we can just fix the `Exchange.sameResult`
to exclude the coordinator?
---
-
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19767
**[Test build #84104 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84104/testReport)**
for PR 19767 at commit
[`86cba3c`](https://github.com/apache/spark/commit/86
Github user inpefess commented on the issue:
https://github.com/apache/spark/pull/15670
I'm also interested in pushing this PR forward. Is there anything else that
is needed to be done to merge it except rebasing it and resolving conflicts?
---
--
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r152526466
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcDeserializer.scala
---
@@ -0,0 +1,226 @@
+/*
+ * Licensed to
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r152525693
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcSerializer.scala
---
@@ -0,0 +1,205 @@
+/*
+ * Licensed to th
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r152525365
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcSerializer.scala
---
@@ -0,0 +1,205 @@
+/*
+ * Licensed to th
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19468#discussion_r152522621
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/KubernetesClusterSchedulerBackend.scala
---
@@ -0,0 +1,429
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19468#discussion_r152505563
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/SparkKubernetesClientFactory.scala
---
@@ -0,0 +1,102 @@
+/*
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19468#discussion_r152521995
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/KubernetesClusterSchedulerBackend.scala
---
@@ -0,0 +1,429
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19468#discussion_r152502008
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/Config.scala
---
@@ -0,0 +1,114 @@
+/*
+ * Licensed to the Ap
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19468#discussion_r152509429
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/ExecutorPodFactory.scala
---
@@ -0,0 +1,226 @@
+/*
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19468#discussion_r152503394
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/Config.scala
---
@@ -0,0 +1,114 @@
+/*
+ * Licensed to the Ap
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r152523925
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcSerializer.scala
---
@@ -0,0 +1,205 @@
+/*
+ * Licensed to th
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r152523189
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcFilters.scala
---
@@ -0,0 +1,183 @@
+/*
+ * Licensed to the A
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r152522983
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcFileFormat.scala
---
@@ -39,3 +58,122 @@ private[sql] object OrcFileF
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r152522244
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcFileFormat.scala
---
@@ -39,3 +58,120 @@ private[sql] object OrcFileF
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r152522180
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcFileFormat.scala
---
@@ -39,3 +58,122 @@ private[sql] object OrcFileF
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r152521905
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcDeserializer.scala
---
@@ -0,0 +1,216 @@
+/*
+ * Licensed to
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r152521387
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcDeserializer.scala
---
@@ -0,0 +1,226 @@
+/*
+ * Licensed to
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r152521192
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcDeserializer.scala
---
@@ -0,0 +1,226 @@
+/*
+ * Licensed to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19791
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19791
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84103/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19791
**[Test build #84103 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84103/testReport)**
for PR 19791 at commit
[`aba0066`](https://github.com/apache/spark/commit/a
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r152518874
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcDeserializer.scala
---
@@ -0,0 +1,216 @@
+/*
+ * Licensed to
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19767#discussion_r152518292
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Expression.scala
---
@@ -105,6 +105,41 @@ abstract class Expression extends T
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19468#discussion_r152518072
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/ExecutorPodFactory.scala
---
@@ -0,0 +1,229 @@
+/*
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19518
let's create a new PR
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19791
**[Test build #84103 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84103/testReport)**
for PR 19791 at commit
[`aba0066`](https://github.com/apache/spark/commit/ab
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19791
test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/19518
ping @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@sp
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19794
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apa
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/19621
@WeichenXu123 with reference to
https://github.com/apache/spark/pull/19621#issuecomment-344530228 - the sort
is
[stable](https://github.com/scala/scala/blob/2.11.x/src/library/scala/collection/SeqL
Github user mpetruska commented on a diff in the pull request:
https://github.com/apache/spark/pull/19791#discussion_r152504268
--- Diff: repl/src/test/scala/org/apache/spark/repl/ReplSuite.scala ---
@@ -217,4 +218,13 @@ class ReplSuite extends SparkFunSuite {
assertDoesNot
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19794
**[Test build #84102 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84102/testReport)**
for PR 19794 at commit
[`951635e`](https://github.com/apache/spark/commit/95
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r152503817
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -140,6 +140,13 @@ object SQLConf {
.booleanConf
GitHub user kiszk opened a pull request:
https://github.com/apache/spark/pull/19794
[SPARK-20101][SQL][FOLLOW-UP] use correct config name
"spark.sql.columnVector.offheap.enabled"
## What changes were proposed in this pull request?
This PR addresses [the spelling
miss](http
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19621
**[Test build #84101 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84101/testReport)**
for PR 19621 at commit
[`031f53f`](https://github.com/apache/spark/commit/03
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/19621
Jenkins retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-ma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19767
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19767
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84099/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19767
**[Test build #84099 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84099/testReport)**
for PR 19767 at commit
[`c875329`](https://github.com/apache/spark/commit/c
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19763#discussion_r152496569
--- Diff: core/src/main/scala/org/apache/spark/MapOutputTracker.scala ---
@@ -472,15 +475,66 @@ private[spark] class MapOutputTrackerMaster(
shuff
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/17436#discussion_r152493744
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -140,6 +140,13 @@ object SQLConf {
.booleanConf
Github user gczsjdy commented on a diff in the pull request:
https://github.com/apache/spark/pull/19763#discussion_r152493779
--- Diff: core/src/main/scala/org/apache/spark/MapOutputTracker.scala ---
@@ -472,15 +475,66 @@ private[spark] class MapOutputTrackerMaster(
shuffle
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19781
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19439
**[Test build #84100 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84100/testReport)**
for PR 19439 at commit
[`aa3c704`](https://github.com/apache/spark/commit/aa
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19781
thanks ,merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19439
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19767
**[Test build #84099 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84099/testReport)**
for PR 19767 at commit
[`c875329`](https://github.com/apache/spark/commit/c8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19786
**[Test build #84098 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84098/testReport)**
for PR 19786 at commit
[`8cfa0b2`](https://github.com/apache/spark/commit/8c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19651
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84097/
Test FAILed.
---
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19786
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19621
**[Test build #84093 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84093/testReport)**
for PR 19621 at commit
[`031f53f`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19651
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19468
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19621
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19621
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84093/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19468
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84096/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19468
**[Test build #84096 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84096/testReport)**
for PR 19468 at commit
[`3b587b4`](https://github.com/apache/spark/commit/3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19651
**[Test build #84097 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84097/testReport)**
for PR 19651 at commit
[`0666d30`](https://github.com/apache/spark/commit/0
201 - 266 of 266 matches
Mail list logo