Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22763
**[Test build #97527 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97527/testReport)**
for PR 22763 at commit
[`9def7e8`](https://github.com/apache/spark/commit/9d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22263
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/22263
**[Test build #97528 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97528/testReport)**
for PR 22263 at commit
[`c7d4b82`](https://github.com/apache/spark/commit/c7
GitHub user mgaido91 opened a pull request:
https://github.com/apache/spark/pull/22764
[SPARK-25765][ML] Add training cost to BisectingKMeans summary
## What changes were proposed in this pull request?
The PR adds the `trainingCost` value to the `BisectingKMeansSummary`, in
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22764
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4085/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22764
**[Test build #97529 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97529/testReport)**
for PR 22764 at commit
[`5919d3c`](https://github.com/apache/spark/commit/59
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22764
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/22762
**[Test build #97526 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97526/testReport)**
for PR 22762 at commit
[`9f7901b`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22764
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/22762
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/22764
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4086/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22762
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97526/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22763
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/22763
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97527/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22763
**[Test build #97527 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97527/testReport)**
for PR 22763 at commit
[`9def7e8`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22677
**[Test build #97518 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97518/testReport)**
for PR 22677 at commit
[`b1f2e48`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22677
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/22677
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97518/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22219
**[Test build #97530 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97530/testReport)**
for PR 22219 at commit
[`136a4f9`](https://github.com/apache/spark/commit/13
Github user Dooyoung-Hwang commented on the issue:
https://github.com/apache/spark/pull/22219
I refactored collectionResultAsSeqView function with using implicit class.
---
-
To unsubscribe, e-mail: reviews-unsubscr.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22762
**[Test build #97531 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97531/testReport)**
for PR 22762 at commit
[`c46693b`](https://github.com/apache/spark/commit/c4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22762
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4087/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22762
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22762
I like it!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/22746#discussion_r226263066
--- Diff: docs/sql-migration-guide-upgrade.md ---
@@ -0,0 +1,520 @@
+---
+layout: global
+title: Spark SQL Upgrading Guide
+displayTitle: Spark
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/22746#discussion_r226262995
--- Diff: docs/sql-migration-guide-upgrade.md ---
@@ -0,0 +1,520 @@
+---
+layout: global
+title: Spark SQL Upgrading Guide
+displayTitle: Spark
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22762
**[Test build #97531 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97531/testReport)**
for PR 22762 at commit
[`c46693b`](https://github.com/apache/spark/commit/c
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22762#discussion_r226264798
--- Diff: python/pyspark/sql/tests.py ---
@@ -225,6 +225,55 @@ def sql_conf(self, pairs):
else:
self.spark.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22762
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/22762
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97531/
Test FAILed.
---
GitHub user lipzhu opened a pull request:
https://github.com/apache/spark/pull/22765
[SPARK-25757][Build] Upgrade netty-all from 4.1.17.Final to 4.1.30.Final
## What changes were proposed in this pull request?
Upgrade netty dependency from 4.1.17 to 4.1.30.
Explanation:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22765
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22756
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/22756
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97525/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22765
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22756
**[Test build #97525 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97525/testReport)**
for PR 22756 at commit
[`d5fddb5`](https://github.com/apache/spark/commit/d
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22762#discussion_r226266575
--- Diff: python/pyspark/sql/tests.py ---
@@ -225,6 +225,55 @@ def sql_conf(self, pairs):
else:
self.spark.conf.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22765
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user lipzhu commented on the issue:
https://github.com/apache/spark/pull/22765
cc @srowen @zsxwing @HyukjinKwon
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-ma
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22762#discussion_r226267706
--- Diff: python/pyspark/sql/tests.py ---
@@ -225,6 +225,55 @@ def sql_conf(self, pairs):
else:
self.spark.
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/22754
Good catch.
One question: can we set `12` into this property?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.or
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22677
**[Test build #97532 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97532/testReport)**
for PR 22677 at commit
[`df21621`](https://github.com/apache/spark/commit/df
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22764
**[Test build #97529 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97529/testReport)**
for PR 22764 at commit
[`5919d3c`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22764
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97529/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22764
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/22743
**[Test build #97522 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97522/testReport)**
for PR 22743 at commit
[`206743c`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22743
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/22743
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97522/
Test PASSed.
---
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22765
That's probably fine.
As an aside, I think we can remove all instances of Netty 3.x in the code
base, if any, now that Flume is gone.
---
---
Github user vofque commented on the issue:
https://github.com/apache/spark/pull/22745
Added these classes:
_UnresolvedGetArrayFromMap_ in unresolved.scala - an unresolved
substitution.
_GetArrayFromMap_ in complexTypeExtractors.scala - an extraction algorithm
(not sure th
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22757
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user gengliangwang commented on the issue:
https://github.com/apache/spark/pull/22663
Retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22756
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apa
Github user gengliangwang commented on the issue:
https://github.com/apache/spark/pull/22663
@dongjoon-hyun Sorry to reply late. I have updated the title and
description.
---
-
To unsubscribe, e-mail: reviews-unsubs
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21164
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22663
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/22663
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4088/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22663
**[Test build #97533 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97533/testReport)**
for PR 22663 at commit
[`e2ca55e`](https://github.com/apache/spark/commit/e2
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22721#discussion_r226280121
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InsertIntoHadoopFsRelationCommand.scala
---
@@ -189,6 +189,7 @@ case class I
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22721
**[Test build #97521 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97521/testReport)**
for PR 22721 at commit
[`983c5a8`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22721
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/22721
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97521/
Test PASSed.
---
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22721
I think it's reasonable to follow `InsertIntoHiveTable`, but it's better to
provide more details about what changes in `InsertIntoHadoopFsRelationCommand`:
1. what's refreshed? Previously we re
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22501
**[Test build #97534 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97534/testReport)**
for PR 22501 at commit
[`82e2367`](https://github.com/apache/spark/commit/82
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22501
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/22501
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4089/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22746
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/22746
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4090/
Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/22746
@kiszk Great thanks for all the detailed check, addressed in 17995f9. Also
double checked by grep the typo for each error you found.
---
--
Github user xuanyuanking commented on a diff in the pull request:
https://github.com/apache/spark/pull/22746#discussion_r226288610
--- Diff: docs/sql-data-sources-other.md ---
@@ -0,0 +1,114 @@
+---
+layout: global
+title: Other Data Sources
+displayTitle: Other Data
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22746
**[Test build #97535 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97535/testReport)**
for PR 22746 at commit
[`17995f9`](https://github.com/apache/spark/commit/17
Github user rvesse commented on a diff in the pull request:
https://github.com/apache/spark/pull/22681#discussion_r226289881
--- Diff:
resource-managers/kubernetes/docker/src/main/dockerfiles/spark/Dockerfile ---
@@ -18,6 +18,7 @@
FROM openjdk:8-alpine
ARG spark_jar
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22762
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/22762
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4091/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22762
**[Test build #97536 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97536/testReport)**
for PR 22762 at commit
[`c405b0e`](https://github.com/apache/spark/commit/c4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22762
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/22762
**[Test build #97536 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97536/testReport)**
for PR 22762 at commit
[`c405b0e`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22762
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97536/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22746
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/22746
**[Test build #97535 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97535/testReport)**
for PR 22746 at commit
[`17995f9`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22762
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4092/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22746
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97535/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22762
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/22762
**[Test build #97537 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97537/testReport)**
for PR 22762 at commit
[`ed25864`](https://github.com/apache/spark/commit/ed
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22749#discussion_r226294017
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/ExpressionEncoder.scala
---
@@ -43,10 +44,11 @@ import org.apache.spark.util
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22749#discussion_r226294255
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/ExpressionEncoder.scala
---
@@ -103,75 +88,61 @@ object ExpressionEncoder {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22749#discussion_r226295859
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/ExpressionEncoder.scala
---
@@ -103,75 +88,61 @@ object ExpressionEncoder {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22749#discussion_r226296369
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/ExpressionEncoder.scala
---
@@ -103,75 +88,61 @@ object ExpressionEncoder {
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20999
Yes, you're right @cloud-fan. I missed that, thanks. I am going to update
this with the required change, but it is going to be a huge change. Thanks.
---
-
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22749#discussion_r226298803
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/ExpressionEncoder.scala
---
@@ -212,21 +183,88 @@ object ExpressionEncoder {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22749#discussion_r226299441
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/ExpressionEncoder.scala
---
@@ -212,21 +183,88 @@ object ExpressionEncoder {
Github user rvesse commented on the issue:
https://github.com/apache/spark/pull/22681
I actually just got bit by this today while trying to run K8S integration
tests with custom images. The integration tests assume the runnable
distribution layout for `/opt/spark/examples` in the ima
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22749#discussion_r226301139
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/ExpressionEncoder.scala
---
@@ -212,21 +183,88 @@ object ExpressionEncoder {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22749#discussion_r226301402
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/ExpressionEncoder.scala
---
@@ -212,21 +183,88 @@ object ExpressionEncoder {
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22749
I like this idea! waiting for tests pass
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comm
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22764
cc @cloud-fan @dongjoon-hyun @holdenk @srowen
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22762
**[Test build #97537 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97537/testReport)**
for PR 22762 at commit
[`ed25864`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22762
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97537/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22762
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22764
does the example need to be updated with this new API?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For a
101 - 200 of 701 matches
Mail list logo