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
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
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
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/22746
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
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
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):
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 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
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
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
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):
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
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
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
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
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 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):
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
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):
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
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
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
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 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
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
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
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
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):
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
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 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:
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:
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:
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 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
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 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 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
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
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
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 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
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
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:
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:
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,
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:
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
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
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 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:
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
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
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 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:
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:
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:
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
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
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):
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:
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
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
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/22677
**[Test build #97518 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97518/testReport)**
for PR 22677 at commit
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
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
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/22764
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
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
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
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
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
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
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):
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 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
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
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
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):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22263
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
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
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22763
cc @cloud-fan @dongjoon-hyun @holdenk @srowen
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/22746#discussion_r226250306
--- Diff: docs/sql-performance-turing.md ---
@@ -0,0 +1,151 @@
+---
+layout: global
+title: Performance Tuning
+displayTitle: Performance
GitHub user mgaido91 opened a pull request:
https://github.com/apache/spark/pull/22763
[SPARK-25764][ML][EXAMPLES] Update BisectingKMeans example not to use
deprecated computeCost method
## What changes were proposed in this pull request?
The PR updates the examples for
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22762
**[Test build #97526 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97526/testReport)**
for PR 22762 at commit
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):
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
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22466
Btw, what if `create database if not exists ...`? Seems like an exception
will be thrown if the table exists even if we specify `if not exists`?
---
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22762
cc @HyukjinKwon
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22762
Jenkins, retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22756
**[Test build #97525 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97525/testReport)**
for PR 22756 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22762
**[Test build #97524 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97524/testReport)**
for PR 22762 at commit
501 - 600 of 709 matches
Mail list logo