Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19338
**[Test build #82215 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82215/testReport)**
for PR 19338 at commit
[`4d906b7`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19338
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/19338
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82215/
Test FAILed.
---
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19349#discussion_r141244651
--- Diff:
core/src/main/scala/org/apache/spark/api/python/PythonRunner.scala ---
@@ -0,0 +1,429 @@
+/*
+ * Licensed to the Apache Software Found
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19346#discussion_r141246595
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala
---
@@ -61,7 +61,36 @@ private[ui] class AllExecutionsPage(paren
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19349#discussion_r141246604
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/python/ArrowPythonRunner.scala
---
@@ -0,0 +1,197 @@
+/*
+ * Licensed to the Apache
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19338
**[Test build #82217 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82217/testReport)**
for PR 19338 at commit
[`05adc2a`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19338
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82217/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19338
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19359
**[Test build #82216 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82216/testReport)**
for PR 19359 at commit
[`5537787`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19349
**[Test build #82214 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82214/testReport)**
for PR 19349 at commit
[`7f6e43f`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19359
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/19349
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/19359
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82216/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19349
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82214/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19337
**[Test build #82219 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82219/testReport)**
for PR 19337 at commit
[`29f05c7`](https://github.com/apache/spark/commit/29
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19338
There's one related test failure, can you please check.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19338
**[Test build #82220 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82220/testReport)**
for PR 19338 at commit
[`9a2d7ba`](https://github.com/apache/spark/commit/9a
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19349#discussion_r141250242
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/python/ArrowPythonRunner.scala
---
@@ -0,0 +1,197 @@
+/*
+ * Licensed to the Apache
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19349#discussion_r141250227
--- Diff: python/pyspark/serializers.py ---
@@ -211,33 +212,37 @@ def __repr__(self):
return "ArrowSerializer"
+def _create_batch
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19349#discussion_r141250251
--- Diff: python/pyspark/serializers.py ---
@@ -251,6 +256,36 @@ def __repr__(self):
return "ArrowPandasSerializer"
+class ArrowS
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19349#discussion_r141250303
--- Diff:
core/src/main/scala/org/apache/spark/api/python/PythonRunner.scala ---
@@ -0,0 +1,429 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19349#discussion_r141250276
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/python/PythonUDFRunner.scala
---
@@ -0,0 +1,103 @@
+/*
+ * Licensed to the Apache S
Github user guoxiaolongzte commented on a diff in the pull request:
https://github.com/apache/spark/pull/19346#discussion_r141251376
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala
---
@@ -61,7 +61,36 @@ private[ui] class AllExecutionsPa
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19346#discussion_r141251711
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala
---
@@ -61,7 +61,36 @@ private[ui] class AllExecutionsPage(paren
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19346#discussion_r141252597
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala
---
@@ -61,7 +61,36 @@ private[ui] class AllExecutionsPage(paren
Github user guoxiaolongzte commented on a diff in the pull request:
https://github.com/apache/spark/pull/19346#discussion_r141252540
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala
---
@@ -61,7 +61,36 @@ private[ui] class AllExecutionsPa
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19346#discussion_r141252776
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala
---
@@ -61,7 +61,36 @@ private[ui] class AllExecutionsPage(paren
Github user guoxiaolongzte commented on a diff in the pull request:
https://github.com/apache/spark/pull/19346#discussion_r141253705
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala
---
@@ -61,7 +61,36 @@ private[ui] class AllExecutionsPa
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19346#discussion_r141253774
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala
---
@@ -61,7 +61,36 @@ private[ui] class AllExecutionsPage(paren
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19349
**[Test build #82221 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82221/testReport)**
for PR 19349 at commit
[`aa3fa70`](https://github.com/apache/spark/commit/aa
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19346
**[Test build #8 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/8/testReport)**
for PR 19346 at commit
[`279e0d2`](https://github.com/apache/spark/commit/27
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19337
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82219/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19337
**[Test build #82219 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82219/testReport)**
for PR 19337 at commit
[`29f05c7`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19337
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/19346
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19349#discussion_r141256118
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/python/ArrowPythonRunner.scala
---
@@ -0,0 +1,197 @@
+/*
+ * Licensed to the Apache
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19349#discussion_r141256430
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/python/ArrowPythonRunner.scala
---
@@ -0,0 +1,197 @@
+/*
+ * Licensed to the Apache
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19184
After discussed with @mridulm offline. Though the patch here cannot address
the issue of `getSortedIterator` - which uses a PriorityQueue, somehow it
solves the problem of `getIterator(...)` which
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19349#discussion_r141257047
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/python/ArrowPythonRunner.scala
---
@@ -0,0 +1,197 @@
+/*
+ * Licensed to the Apache
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19349#discussion_r141257653
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/python/ArrowPythonRunner.scala
---
@@ -0,0 +1,197 @@
+/*
+ * Licensed to the Apache
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/19349
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache
401 - 442 of 442 matches
Mail list logo