Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22773
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22766
**[Test build #97603 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97603/testReport)**
for PR 22766 at commit
[`6e6eca4`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22766
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/22766
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97603/
Test PASSed.
---
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22743
Datasource table will not cache in
[tableRelationCache](https://github.com/apache/spark/blob/01c3dfab158d40653f8ce5d96f57220297545d5b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22774
**[Test build #97596 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97596/testReport)**
for PR 22774 at commit
[`7076bde`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22774
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/22774
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97596/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22775
**[Test build #97605 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97605/testReport)**
for PR 22775 at commit
[`2705aa8`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22775
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97605/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22775
Merged build finished. Test FAILed.
---
-
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/22775
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22775
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/22775
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/4128/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22721
**[Test build #97597 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97597/testReport)**
for PR 22721 at commit
[`6c8a73f`](https://github.com/apache/spark/commit/6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22775
**[Test build #97610 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97610/testReport)**
for PR 22775 at commit
[`2705aa8`](https://github.com/apache/spark/commit/27
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/97597/
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 comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22732
**[Test build #97601 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97601/testReport)**
for PR 22732 at commit
[`cb7e97a`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22732
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97601/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22732
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/22466
**[Test build #97602 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97602/testReport)**
for PR 22466 at commit
[`28a086d`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22466
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/22466
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97602/
Test PASSed.
---
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/22750
I thought about the last line
> This PR removes ColumnarBatchScan.needsUnsafeRowConversion, and keep this
flag only in FileSourceScanExec
---
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/22774
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/22774
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/4129/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22774
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/22774
**[Test build #97611 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97611/testReport)**
for PR 22774 at commit
[`7076bde`](https://github.com/apache/spark/commit/70
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22773
**[Test build #97604 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97604/testReport)**
for PR 22773 at commit
[`3447e73`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22773
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97604/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22773
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/22429
**[Test build #97600 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97600/testReport)**
for PR 22429 at commit
[`9f1d11d`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22429
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97600/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22429
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user RussellSpitzer commented on a diff in the pull request:
https://github.com/apache/spark/pull/22576#discussion_r226622273
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/SparkSessionExtensions.scala ---
@@ -168,4 +173,21 @@ class SparkSessionExtensions {
de
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/22576#discussion_r226623886
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/SparkSessionExtensions.scala ---
@@ -168,4 +173,21 @@ class SparkSessionExtensions {
def in
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22466#discussion_r226624218
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -207,6 +207,14 @@ class SessionCatalog(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22666
**[Test build #97607 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97607/testReport)**
for PR 22666 at commit
[`41c39db`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22666
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97607/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22666
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/22776
**[Test build #97609 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97609/testReport)**
for PR 22776 at commit
[`7f4cd6d`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22776
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/22776
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97609/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22482
**[Test build #97608 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97608/testReport)**
for PR 22482 at commit
[`dd29af2`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22482
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97608/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22482
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 #97606 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97606/testReport)**
for PR 22263 at commit
[`e2b5dcf`](https://github.com/apache/spark/commit/e
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22750
`DataSourceScanExec` does not have `needsUnsafeRowConversion`
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.or
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):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97606/
Test PASSed.
---
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 cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22732
thanks, merging to master/2.4!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-ma
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22732
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22666#discussion_r226640362
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/csvExpressions.scala
---
@@ -60,7 +63,7 @@ case class CsvToStructs(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22666#discussion_r226640860
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CsvExpressionsSuite.scala
---
@@ -155,4 +155,15 @@ class CsvExpressionsSu
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22666#discussion_r226641023
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -3886,6 +3886,31 @@ object functions {
withExpr(new CsvToStructs(e.e
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22743
> Datasource table will not cache in tableRelationCache.
I don't think so. Spark caches data source table in `FindDataSourceTable`
---
---
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/22146#discussion_r226642833
--- Diff: docs/running-on-kubernetes.md ---
@@ -799,4 +815,168 @@ specific to Spark on Kubernetes.
This sets the major Python version of the docker im
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/22146#discussion_r226643416
--- Diff: docs/running-on-kubernetes.md ---
@@ -799,4 +815,168 @@ specific to Spark on Kubernetes.
This sets the major Python version of the docker im
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22766
thanks, merging to master/2.4/2.3!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22766
---
-
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/22598
**[Test build #97612 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97612/testReport)**
for PR 22598 at commit
[`7c22433`](https://github.com/apache/spark/commit/7c
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22743
Yes. you are right, if datasource table stats is empty,
`DetermineTableStats` doesn't set stats for it, so it's only a problem for hive
tables.
---
---
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/22771
the change itself seems reasonable we were actually looking at this
recently as there are a couple of others bugs around this:
https://issues.apache.org/jira/browse/SPARK-25250 and
https://issues
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/22771#discussion_r226651308
--- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
---
@@ -1364,6 +1385,16 @@ private[spark] class DAGScheduler(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22598
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97612/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22598
**[Test build #97612 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97612/testReport)**
for PR 22598 at commit
[`7c22433`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22598
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22764#discussion_r226652377
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeansModel.scala
---
@@ -225,13 +227,14 @@ object BisectingKMeansModel extends
Github user rvesse commented on the issue:
https://github.com/apache/spark/pull/22748
Rebased onto master, should be ready for merging
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22466#discussion_r226655438
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -207,6 +207,14 @@ class SessionCatalog(
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r226659261
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/TokenUtil.scala
---
@@ -0,0 +1,111 @@
+/*
+ * Licensed to the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22775
**[Test build #97610 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97610/testReport)**
for PR 22775 at commit
[`2705aa8`](https://github.com/apache/spark/commit/2
Github user gaborgsomogyi commented on the issue:
https://github.com/apache/spark/pull/22598
retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22598
**[Test build #97613 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97613/testReport)**
for PR 22598 at commit
[`7c22433`](https://github.com/apache/spark/commit/7c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22775
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/22775
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97610/
Test FAILed.
---
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/22771
Actually as part of the jiras I mentioned above we were looking at killing
other task attempts as soon as one task attempt succeeds rather then waiting
for the entire job to finish, thoughts on t
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22598
**[Test build #97613 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97613/testReport)**
for PR 22598 at commit
[`7c22433`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22598
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97613/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22598
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22764#discussion_r226668414
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeansModel.scala
---
@@ -225,13 +227,14 @@ object BisectingKMeansModel extends
Github user sandeep-katta commented on a diff in the pull request:
https://github.com/apache/spark/pull/22466#discussion_r226673513
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -207,6 +207,14 @@ class SessionCatalog(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22466
**[Test build #97614 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97614/testReport)**
for PR 22466 at commit
[`b091996`](https://github.com/apache/spark/commit/b0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22466
**[Test build #97615 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97615/testReport)**
for PR 22466 at commit
[`570c5b7`](https://github.com/apache/spark/commit/57
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/22764#discussion_r226688846
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeansModel.scala
---
@@ -225,13 +227,14 @@ object BisectingKMeansModel extends
L
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22774
**[Test build #97611 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97611/testReport)**
for PR 22774 at commit
[`7076bde`](https://github.com/apache/spark/commit/7
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22764#discussion_r226693546
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeansModel.scala
---
@@ -225,13 +227,14 @@ object BisectingKMeansModel extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22774
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/22774
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97611/
Test PASSed.
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22263
Thank you, @wangyum , @cloud-fan , @maropu , @mgaido91 !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/22295
I'll leave this for if @HyukjinKwon has any final comments, otherwise I'm
happy to merge.
---
-
To unsubscribe, e-mail: reviews-
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22263
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/21157
If removing the hack entirely is going to brake named tuples defined in the
repl I'm a -1 on that change. While we certainly are more free to make breaking
API changes in a majour version release we
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22504#discussion_r226706867
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -800,14 +817,33 @@ private[history] class FsHistoryProvider(conf
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/22533
Jenkins OK to test.
@annamolchanova if you want help making the Scala version of this PR first
I'd be happy to lend what help I can.
---
---
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/18457
Would you be OK closing this PR @SoulGuedria
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional c
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/22425
Gentle ping, whats up?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user ankuriitg commented on a diff in the pull request:
https://github.com/apache/spark/pull/22504#discussion_r226710526
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -800,14 +817,33 @@ private[history] class FsHistoryProvider(c
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22775
The latest Python failure looks relevant to this PR.
```scala
AnalysisException: u"cannot resolve 'schemaofjson(`value`)' due to data
type mismatch: The input json should be a string li
101 - 200 of 446 matches
Mail list logo