Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23103
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/5267/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23103
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/23103
**[Test build #99167 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99167/testReport)**
for PR 23103 at commit
[`39424dd`](https://github.com/apache/spark/commit/39
Github user zouzias commented on a diff in the pull request:
https://github.com/apache/spark/pull/23103#discussion_r235627877
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaSourceProvider.scala
---
@@ -538,6 +538,17 @@ private[kafka010] object
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/23027
I like new benchmark, but I still hope we have a separate suite for
in-memory benchmark.
The current one looks misleading and inconsistent in this file.
---
-
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/23101
Welcome, @tasanuma !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: re
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22309
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user tasanuma commented on the issue:
https://github.com/apache/spark/pull/23101
Thanks a lot!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/23101
---
-
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/22979
**[Test build #99166 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99166/testReport)**
for PR 22979 at commit
[`2918d04`](https://github.com/apache/spark/commit/29
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/23101
Thank you all. Merged to `master`.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comman
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/23115
This is merged to `branch-2.2`. Please close this PR.
Thanks, @attilapiros .
---
-
To unsubscribe, e-mail: reviews-unsu
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/22979#discussion_r235618859
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/csv/CSVExprUtils.scala
---
@@ -79,4 +83,22 @@ object CSVExprUtils {
throw
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/23114
This is merged now. Please close this PR.
Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Fo
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/23114
Merged to `branch-2.3`.
---
-
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/23111
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99163/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23111
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/23111
**[Test build #99163 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99163/testReport)**
for PR 23111 at commit
[`d3db950`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23116
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/23116
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/5266/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23116
**[Test build #99165 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99165/testReport)**
for PR 23116 at commit
[`dcd517d`](https://github.com/apache/spark/commit/dc
GitHub user jerryshao opened a pull request:
https://github.com/apache/spark/pull/23116
[SPARK-24553][UI][FOLLOWUP] Fix unnecessary UI redirect
## What changes were proposed in this pull request?
This PR is a follow-up PR of #21600 to fix the unnecessary UI redirect.
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/23115
Thank you, @attilapiros . I'll do the manual test.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For a
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/23114
Apache Spark merge script removes '@' from the commit message in order to
avoid pinging people.
---
-
To unsubscribe, e-ma
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/23114
Oh, I got it. The PR description is wrong in this PR.
The current one.
```
$ curl -H cookie http://458apiros-MBP.lan:18080/
```
The correct one needs '@'.
```
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23111
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/5265/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23111
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/23111
**[Test build #99164 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99164/testReport)**
for PR 23111 at commit
[`973f6da`](https://github.com/apache/spark/commit/97
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/23114
@attilapiros . I followed the verification steps in the PR description,
but I cannot reproduce this issue at Spark 2.4.0 and 2.3.2 History Server. I'm
using Mac like the following. Is there s
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23111
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/23111
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/5264/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23111
**[Test build #99163 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99163/testReport)**
for PR 23111 at commit
[`d3db950`](https://github.com/apache/spark/commit/d3
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23111
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/23109
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/5263/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23109
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/23109
**[Test build #99162 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99162/testReport)**
for PR 23109 at commit
[`91193e7`](https://github.com/apache/spark/commit/91
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23109
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/23115
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99157/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23115
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/23115
**[Test build #99157 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99157/testReport)**
for PR 23115 at commit
[`9513b3f`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
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/23100
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99152/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23114
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99153/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23114
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/23100
**[Test build #99152 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99152/testReport)**
for PR 23100 at commit
[`40de38f`](https://github.com/apache/spark/commit/4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23114
**[Test build #99153 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99153/testReport)**
for PR 23114 at commit
[`01bd63b`](https://github.com/apache/spark/commit/0
Github user wypoon commented on a diff in the pull request:
https://github.com/apache/spark/pull/23058#discussion_r235603798
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -693,9 +693,9 @@ private[spark] class BlockManager(
*/
privat
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23101
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99150/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23101
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/23101
**[Test build #99150 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99150/testReport)**
for PR 23101 at commit
[`f10729a`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
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/5262/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
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/23100
**[Test build #99161 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99161/testReport)**
for PR 23100 at commit
[`64364b9`](https://github.com/apache/spark/commit/64
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21688
Test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/23100
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/23043
`org.apache.spark.sql.MathFunctionsSuite.rint` seems failed due to this
change. Can you check it?
---
-
To unsubscribe, e-mail: r
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23043
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99155/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23043
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/23043
**[Test build #99155 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99155/testReport)**
for PR 23043 at commit
[`a07e614`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99154/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23100
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/23100
**[Test build #99154 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99154/testReport)**
for PR 23100 at commit
[`64364b9`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23111
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/23111
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99159/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23111
**[Test build #99159 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99159/testReport)**
for PR 23111 at commit
[`d3db950`](https://github.com/apache/spark/commit/d
Github user heary-cao commented on the issue:
https://github.com/apache/spark/pull/23084
so this PR is just code cleanup, not a real bug. thanks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23055
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99148/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23055
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/23055
**[Test build #99148 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99148/testReport)**
for PR 23055 at commit
[`fd92a4e`](https://github.com/apache/spark/commit/f
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/23043#discussion_r235596825
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameAggregateSuite.scala ---
@@ -723,4 +723,18 @@ class DataFrameAggregateSuite extends QueryTest
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23112
**[Test build #99160 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99160/testReport)**
for PR 23112 at commit
[`2deac2a`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23112
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/23112
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99160/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23112
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/23112
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/5261/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23112
**[Test build #99160 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99160/testReport)**
for PR 23112 at commit
[`2deac2a`](https://github.com/apache/spark/commit/2d
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23112
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23112
Looks okay to go.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23111
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/5260/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23111
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/23111
**[Test build #99159 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99159/testReport)**
for PR 23111 at commit
[`d3db950`](https://github.com/apache/spark/commit/d3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23111
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99158/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23111
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/23111
**[Test build #99158 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99158/testReport)**
for PR 23111 at commit
[`29d15d8`](https://github.com/apache/spark/commit/29
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16732
**[Test build #4438 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4438/testReport)**
for PR 16732 at commit
[`540b261`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23111
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/23111
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99156/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23111
**[Test build #99156 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99156/testReport)**
for PR 23111 at commit
[`ec0f730`](https://github.com/apache/spark/commit/e
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/23070
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23109
retest this please
(the test failures look not persistent)
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.a
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23070
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user carsonwang commented on the issue:
https://github.com/apache/spark/pull/22907
What if there is a FetchFailure and Spark reruns some tasks in the previous
succeeded shuffle map stage? That will be a new ShuffleMapStage and we will
still double counting the accumulators, rig
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/23054
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23080
LGTM except https://github.com/apache/spark/pull/23080/files#r235589426
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23109
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99147/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23109
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/23080#discussion_r235589426
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/csv/CSVOptions.scala
---
@@ -216,8 +232,13 @@ class CSVOptions(
format.set
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23109
**[Test build #99147 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99147/testReport)**
for PR 23109 at commit
[`91193e7`](https://github.com/apache/spark/commit/9
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/23080#discussion_r235589448
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/csv/CSVOptions.scala
---
@@ -227,7 +248,10 @@ class CSVOptions(
settings.s
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23054
thanks, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
1 - 100 of 604 matches
Mail list logo