Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16981
**[Test build #73815 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73815/testReport)**
for PR 16981 at commit
[`ddc06cf`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16910
**[Test build #73829 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73829/testReport)**
for PR 16910 at commit
[`15c0a77`](https://github.com/apache/spark/commit/15
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17081
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is ena
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17081
thanks, merging to master!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17096
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73822/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17096
**[Test build #73822 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73822/testReport)**
for PR 17096 at commit
[`cd235a7`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17096
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17147
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17147
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73826/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17147
**[Test build #73826 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73826/testReport)**
for PR 17147 at commit
[`22aa879`](https://github.com/apache/spark/commit/2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16944
**[Test build #73828 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73828/testReport)**
for PR 16944 at commit
[`95af481`](https://github.com/apache/spark/commit/95
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17001
**[Test build #73827 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73827/testReport)**
for PR 17001 at commit
[`e3a467e`](https://github.com/apache/spark/commit/e3
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17096
@holdenk and @viirya, I got rid of the changes in `types.py` and only left
that I am pretty sure.
There are two kind of changes here that look used in the only local scope.
One
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17136
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73816/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17136
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17136
**[Test build #73816 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73816/testReport)**
for PR 17136 at commit
[`5a01a9d`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17122
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17122
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73813/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17147
**[Test build #73826 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73826/testReport)**
for PR 17147 at commit
[`22aa879`](https://github.com/apache/spark/commit/22
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17122
**[Test build #73813 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73813/testReport)**
for PR 17122 at commit
[`7f095c0`](https://github.com/apache/spark/commit/7
Github user budde commented on the issue:
https://github.com/apache/spark/pull/16944
Retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user windpiger commented on a diff in the pull request:
https://github.com/apache/spark/pull/17001#discussion_r104101806
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveSparkSubmitSuite.scala ---
@@ -905,3 +934,91 @@ object SPARK_18989_DESC_TABLE {
}
GitHub user jerryshao opened a pull request:
https://github.com/apache/spark/pull/17147
[Minor][Doc] Fix doc for web UI https configuration
## What changes were proposed in this pull request?
Doc about enabling web UI https is not correct, "spark.ui.https.enabled" is
not ex
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17145
**[Test build #73825 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73825/testReport)**
for PR 17145 at commit
[`f5a35f6`](https://github.com/apache/spark/commit/f5
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16696#discussion_r104101253
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/StatisticsCollectionSuite.scala ---
@@ -116,22 +116,22 @@ class StatisticsCollectionSuite extends
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/17145
unrelated failure: `
org.apache.spark.sql.kafka010.KafkaSourceStressForDontFailOnDataLossSuite.stress
test for failOnDataLoss=false`. retest this please.
---
If your project is set up for it, you
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16696#discussion_r104101031
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/StatsEstimationSuite.scala
---
@@ -0,0 +1,121 @@
+/*
+ * Lice
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17094
**[Test build #73823 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73823/testReport)**
for PR 17094 at commit
[`d7dceeb`](https://github.com/apache/spark/commit/d7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16696
**[Test build #73824 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73824/testReport)**
for PR 16696 at commit
[`5692939`](https://github.com/apache/spark/commit/56
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17096
**[Test build #73822 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73822/testReport)**
for PR 17096 at commit
[`cd235a7`](https://github.com/apache/spark/commit/cd
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16696#discussion_r104100931
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/StatsConfSuite.scala
---
@@ -1,64 +0,0 @@
-/*
- * Licensed to
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/17135
I remember FileSystem will be cached internally by default. Closing it
probably will introduce some performance regression.
---
If your project is set up for it, you can reply to this email and hav
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/17094
Removed WIP, think it's ready now :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17145
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73817/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17145
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17145
**[Test build #73817 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73817/testReport)**
for PR 17145 at commit
[`f5a35f6`](https://github.com/apache/spark/commit/f
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16696
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17094
**[Test build #73821 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73821/testReport)**
for PR 17094 at commit
[`76eda69`](https://github.com/apache/spark/commit/76
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17094
**[Test build #73820 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73820/testReport)**
for PR 17094 at commit
[`46630d1`](https://github.com/apache/spark/commit/46
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17094
**[Test build #73819 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73819/testReport)**
for PR 17094 at commit
[`f7e9169`](https://github.com/apache/spark/commit/f7
Github user witgo commented on the issue:
https://github.com/apache/spark/pull/15505
[SPARK-18890_20170303](https://github.com/witgo/spark/commits/SPARK-18890_20170303)
`s code is older but the test case running time is 5.2 s
---
If your project is set up for it, you can reply to t
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17096
Let me check if each is fine for sure.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17096
@viirya, thank you so much for taking a look and your time.
So, basically, the second case it compares str to unicode as below:
```python
>>> u"測試" == u"測試".encode("
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17065#discussion_r104098256
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -95,15 +84,16 @@ case class F
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16981
**[Test build #73818 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73818/testReport)**
for PR 16981 at commit
[`4efae36`](https://github.com/apache/spark/commit/4e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16981
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73809/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16981
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16981
**[Test build #73809 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73809/testReport)**
for PR 16981 at commit
[`0d087b0`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17074
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73805/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17074
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user jerryshao closed the pull request at:
https://github.com/apache/spark/pull/14789
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17095#discussion_r104095925
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -1843,10 +1843,12 @@ class DDLSuite extends QueryTest wit
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/14731
@srowen Waiting for your final OK
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/17117#discussion_r104084997
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/KMeans.scala
---
@@ -253,7 +255,18 @@ object KMeansModel extends MLReadable[KMeansModel] {
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/17117#discussion_r104084877
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/KMeans.scala
---
@@ -123,7 +126,8 @@ class KMeansModel private[ml] (
@Since("2.0.0")
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/17117#discussion_r104095197
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/clustering/KMeansSuite.scala ---
@@ -182,6 +224,7 @@ object KMeansSuite {
"predictionCol" -> "myP
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/17117#discussion_r104091867
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/KMeans.scala ---
@@ -418,6 +418,8 @@ object KMeans {
val RANDOM = "random"
@Si
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/17117#discussion_r104092158
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/clustering/KMeansSuite.scala ---
@@ -22,22 +22,28 @@ import scala.util.Random
import org.apache.spark
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/17117#discussion_r104090529
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/KMeans.scala
---
@@ -337,15 +366,61 @@ class KMeans @Since("1.5.0") (
@Since("1.5.
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/17117#discussion_r104094526
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/util/DefaultReadWriteTest.scala ---
@@ -111,12 +113,20 @@ trait DefaultReadWriteTest extends TempDirectory
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/17117#discussion_r104090273
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/KMeans.scala
---
@@ -337,15 +366,61 @@ class KMeans @Since("1.5.0") (
@Since("1.5.
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/17117#discussion_r104092773
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/clustering/KMeansSuite.scala ---
@@ -152,6 +158,35 @@ class KMeansSuite extends SparkFunSuite with
MLlibT
Github user witgo commented on the issue:
https://github.com/apache/spark/pull/15505
Yes, maybe a multithreaded serialization task code can have a better
performance, let me close the PR
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user witgo closed the pull request at:
https://github.com/apache/spark/pull/15505
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enab
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17133
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73807/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17133
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17133
**[Test build #73807 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73807/testReport)**
for PR 17133 at commit
[`37f26e3`](https://github.com/apache/spark/commit/3
Github user skambha commented on the issue:
https://github.com/apache/spark/pull/17067
Thanks a lot Xiao.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so,
Github user imatiach-msft commented on the issue:
https://github.com/apache/spark/pull/16883
@VinceShieh I added some minor comments. This is a nice feature!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your projec
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16883#discussion_r104094424
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/StringIndexer.scala ---
@@ -163,25 +187,28 @@ class StringIndexerModel (
}
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16883#discussion_r104093892
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/StringIndexer.scala ---
@@ -105,7 +125,11 @@ class StringIndexer @Since("1.4.0") (
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/13320
@gatorsmile Could you check this and give me comments, too?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not ha
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16883#discussion_r104093629
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/StringIndexer.scala ---
@@ -163,25 +187,28 @@ class StringIndexerModel (
}
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17145
**[Test build #73817 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73817/testReport)**
for PR 17145 at commit
[`f5a35f6`](https://github.com/apache/spark/commit/f5
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16883#discussion_r104093452
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/StringIndexer.scala ---
@@ -163,25 +190,28 @@ class StringIndexerModel (
}
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16883#discussion_r104093159
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/StringIndexer.scala ---
@@ -105,7 +125,11 @@ class StringIndexer @Since("1.4.0") (
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16883#discussion_r104093069
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/StringIndexer.scala ---
@@ -71,18 +92,17 @@ class StringIndexer @Since("1.4.0") (
de
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16883#discussion_r104092772
--- Diff: docs/ml-features.md ---
@@ -576,7 +579,22 @@ will be generated:
2 | c| 1.0
-Notice that the row containing
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17136
**[Test build #73816 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73816/testReport)**
for PR 17136 at commit
[`5a01a9d`](https://github.com/apache/spark/commit/5a
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16883#discussion_r104092723
--- Diff: docs/ml-features.md ---
@@ -576,7 +579,22 @@ will be generated:
2 | c| 1.0
-Notice that the row containing
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16944
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16883#discussion_r104092627
--- Diff: docs/ml-features.md ---
@@ -542,12 +543,13 @@ column, we should get the following:
"a" gets index `0` because it is the most frequent, f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16944
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73808/
Test FAILed.
---
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/15928
@rxin yea, I got x1.3-1.4 performance gains in this pr.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have t
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16944
**[Test build #73808 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73808/testReport)**
for PR 16944 at commit
[`514ae06`](https://github.com/apache/spark/commit/5
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15928
What do you mean? The improvement was small?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17136
Jenkins, retest this please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wis
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/15928
I looked into this though, I got a little luck from this fix. So, I'll
close for now. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub a
Github user maropu closed the pull request at:
https://github.com/apache/spark/pull/15928
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is ena
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17140
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73802/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17140
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/16981
@gatorsmile okay, I'll fix the issues you mentioned.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16981
**[Test build #73815 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73815/testReport)**
for PR 16981 at commit
[`ddc06cf`](https://github.com/apache/spark/commit/dd
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17140
**[Test build #73802 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73802/testReport)**
for PR 17140 at commit
[`99692bf`](https://github.com/apache/spark/commit/9
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/17122#discussion_r104091814
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -206,6 +206,18 @@ trait CodegenSupport extends SparkPlan
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/16981#discussion_r104091757
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonUtils.scala
---
@@ -55,4 +60,22 @@ object JacksonUtils {
sche
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/16981#discussion_r104091471
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonUtils.scala
---
@@ -55,4 +60,22 @@ object JacksonUtils {
sche
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/16981#discussion_r104091422
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -3007,7 +3008,7 @@ object functions {
* @since 2.1.0
*/
d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17144
**[Test build #73814 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73814/testReport)**
for PR 17144 at commit
[`9ec5caf`](https://github.com/apache/spark/commit/9e
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16981#discussion_r104091265
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/JsonFunctionsSuite.scala ---
@@ -174,4 +174,22 @@ class JsonFunctionsSuite extends QueryTest with
1 - 100 of 671 matches
Mail list logo