Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18659
LGTM, merging to master!
We can address remaining minor comments in follow-up, and have new PRs to
remove the 0-parameter UDF and use arrow streaming protocol.
---
-
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19302
**[Test build #82074 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82074/testReport)**
for PR 19302 at commit
[`795f41a`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19302
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82074/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19302
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/19302
**[Test build #82074 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82074/testReport)**
for PR 19302 at commit
[`795f41a`](https://github.com/apache/spark/commit/79
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19302
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/19301
Regarding performance regression, I think you should post benchmark numbers.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spa
Github user kevinyu98 commented on the issue:
https://github.com/apache/spark/pull/19302
@srowen sure, I will open a new JIRA for this.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additiona
Github user kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19302#discussion_r140433877
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -533,20 +533,30 @@ object StringTrim {
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/19303
yes,it is no problemï¼this is just an optimization.
You are right,it is standalone mode
---
-
To unsubscribe, e-mail: revie
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18805
**[Test build #3930 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3930/testReport)**
for PR 18805 at commit
[`029a753`](https://github.com/apache/spark/commit/0
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/19321
cc @cloud-fan @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19321
**[Test build #82073 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82073/testReport)**
for PR 19321 at commit
[`958715b`](https://github.com/apache/spark/commit/95
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/19321#discussion_r140429643
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/aggregate/ApproximatePercentileSuite.scala
---
@@ -270,7 +270,6 @@ class Appr
GitHub user wzhfy opened a pull request:
https://github.com/apache/spark/pull/19321
[SPARK-22100] [SQL] Make percentile_approx support numeric/date/timestamp
types
## What changes were proposed in this pull request?
Currently `percentile_approx` only supports double type.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19277
**[Test build #3929 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3929/testReport)**
for PR 19277 at commit
[`e3f8e0d`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19319
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82070/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19319
Merged build finished. Test PASSed.
---
-
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/19319
**[Test build #82070 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82070/testReport)**
for PR 19319 at commit
[`779eb40`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19277
**[Test build #3929 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3929/testReport)**
for PR 19277 at commit
[`e3f8e0d`](https://github.com/apache/spark/commit/e
Github user animenon commented on the issue:
https://github.com/apache/spark/pull/19315
Right, but if you want to check a few things together like "who", "who's",
"why", "when", etc you will have to do many finds; but in an ordered document
it would be much easier.
---
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/19320
@HyukjinKwon @jerryshao @ajbozarth
Help to review the code, thanks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/19315
Just ctrl-F and find?
---
-
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/18015
**[Test build #82072 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82072/testReport)**
for PR 18015 at commit
[`21e2c31`](https://github.com/apache/spark/commit/21
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19312
---
-
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/19312
**[Test build #82071 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82071/testReport)**
for PR 19312 at commit
[`94d7659`](https://github.com/apache/spark/commit/94
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18015
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19312#discussion_r140425239
--- Diff: dev/create-release/release-build.sh ---
@@ -95,6 +95,28 @@ if [ -z "$SPARK_VERSION" ]; then
| grep -v INFO | grep -v WARNING | grep -v Dow
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19319
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82069/
Test FAILed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18659#discussion_r140424910
--- Diff: python/pyspark/serializers.py ---
@@ -199,6 +211,55 @@ def __repr__(self):
return "ArrowSerializer"
+class ArrowPand
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/18015
This Jenkins error is not caused by this PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19319
**[Test build #82070 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82070/testReport)**
for PR 19319 at commit
[`779eb40`](https://github.com/apache/spark/commit/77
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/19319
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18015
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82064/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19319
**[Test build #82069 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82069/testReport)**
for PR 19319 at commit
[`779eb40`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18015
**[Test build #82064 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82064/testReport)**
for PR 18015 at commit
[`21e2c31`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19319
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/18015
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18994
Before we review the DDL changes, we need to see the PRs that can get
benefits from this.
---
-
To unsubscribe, e-mail: revi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19319
**[Test build #82068 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82068/testReport)**
for PR 19319 at commit
[`e12f576`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19319
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82068/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19319
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/10949
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
301 - 343 of 343 matches
Mail list logo