Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14788
**[Test build #66626 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66626/consoleFull)**
for PR 14788 at commit
[`8c50b2c`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14788
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14788
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66626/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12775
**[Test build #66624 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66624/consoleFull)**
for PR 12775 at commit
[`699730b`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12775
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/12775
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66624/
Test PASSed.
---
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/13775
This is an interesting pull request (very similar to the Parquet approach).
Two high level comments:
1. Can we just convert Hive's vector format into our own column batch, to
avoid turning the
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14702
@tejasapatil I was checking with @hvanhovell. We should merge this one
soon. Mind bringing it up to date?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15212
**[Test build #66633 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66633/consoleFull)**
for PR 15212 at commit
[`e141c68`](https://github.com/apache/spark/commit/e
Github user bkpathak commented on the issue:
https://github.com/apache/spark/pull/15409
Hi @rxin, I removed those lines and fixed the errors. @srinathshankar
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/15415
**[Test build #66630 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66630/consoleFull)**
for PR 15415 at commit
[`2f1a08c`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15415
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/15409
**[Test build #3303 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3303/consoleFull)**
for PR 15409 at commit
[`bf18321`](https://github.com/apache/spark/commit/
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15409
Thanks!
---
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 feat
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15415
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66630/
Test PASSed.
---
GitHub user yangw1234 opened a pull request:
https://github.com/apache/spark/pull/15416
[SPARK-17849] Fix NPE problem when using grouping sets
## What changes were proposed in this pull request?
Prior this pr, the following code would cause an NPE:
`case class point(a:String,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15212
**[Test build #66634 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66634/consoleFull)**
for PR 15212 at commit
[`d05d7de`](https://github.com/apache/spark/commit/d
Github user yangw1234 commented on the issue:
https://github.com/apache/spark/pull/15416
cc @davies Would you help reviewing this?
---
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 rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r82554551
--- Diff:
core/src/main/java/org/apache/spark/io/NioBasedBufferedFileInputStream.java ---
@@ -0,0 +1,120 @@
+/*
+ * Licensed under the Apache License, V
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r82554563
--- Diff:
core/src/main/java/org/apache/spark/io/NioBasedBufferedFileInputStream.java ---
@@ -0,0 +1,120 @@
+/*
+ * Licensed under the Apache License, V
Github user yangw1234 commented on the issue:
https://github.com/apache/spark/pull/15416
also cc @hvanhovell
---
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 s
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r82554812
--- Diff:
core/src/main/java/org/apache/spark/io/NioBasedBufferedFileInputStream.java ---
@@ -0,0 +1,120 @@
+/*
+ * Licensed under the Apache License, V
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15416
Can one of the admins verify this patch?
---
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 feat
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r82554908
--- Diff:
core/src/main/java/org/apache/spark/io/NioBasedBufferedFileInputStream.java ---
@@ -0,0 +1,120 @@
+/*
+ * Licensed under the Apache License, V
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r82555067
--- Diff:
core/src/main/java/org/apache/spark/io/NioBasedBufferedFileInputStream.java ---
@@ -0,0 +1,120 @@
+/*
+ * Licensed under the Apache License, V
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r82555106
--- Diff:
core/src/main/java/org/apache/spark/io/NioBasedBufferedFileInputStream.java ---
@@ -0,0 +1,120 @@
+/*
+ * Licensed under the Apache License, V
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r82555360
--- Diff:
core/src/main/java/org/apache/spark/io/NioBasedBufferedFileInputStream.java ---
@@ -0,0 +1,120 @@
+/*
+ * Licensed under the Apache License, V
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15292
**[Test build #66628 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66628/consoleFull)**
for PR 15292 at commit
[`350f55d`](https://github.com/apache/spark/commit/
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r82555460
--- Diff:
core/src/main/java/org/apache/spark/io/NioBasedBufferedFileInputStream.java ---
@@ -0,0 +1,120 @@
+/*
+ * Licensed under the Apache License, V
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r82555428
--- Diff:
core/src/main/java/org/apache/spark/io/NioBasedBufferedFileInputStream.java ---
@@ -0,0 +1,120 @@
+/*
+ * Licensed under the Apache License, V
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r8223
--- Diff:
core/src/main/java/org/apache/spark/io/NioBasedBufferedFileInputStream.java ---
@@ -0,0 +1,120 @@
+/*
+ * Licensed under the Apache License, V
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15292
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66628/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15292
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 rxin commented on the issue:
https://github.com/apache/spark/pull/15408
Looks pretty good overall. I left some comments mostly on styling,
documentation, and test cases.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r82556289
--- Diff:
core/src/main/java/org/apache/spark/io/NioBasedBufferedFileInputStream.java ---
@@ -0,0 +1,120 @@
+/*
+ * Licensed under the Apache License,
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14897
---
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 srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r82556320
--- Diff:
core/src/main/java/org/apache/spark/io/NioBasedBufferedFileInputStream.java ---
@@ -0,0 +1,120 @@
+/*
+ * Licensed under the Apache License,
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r82556487
--- Diff:
core/src/test/java/org/apache/spark/io/NioBasedBufferedFileInputStreamSuite.java
---
@@ -0,0 +1,102 @@
+/*
+ * Licensed to the Apache Softw
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14897
thanks for the review, 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
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15416#discussion_r82556934
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -2189,6 +2189,24 @@ class SQLQuerySuite extends QueryTest with
SharedSQLConte
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15386
@BryanCutler if you'll update accordingly I'll merge it.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15409
**[Test build #3303 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3303/consoleFull)**
for PR 15409 at commit
[`bf18321`](https://github.com/apache/spark/commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15414
**[Test build #66635 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66635/consoleFull)**
for PR 15414 at commit
[`6c2a8d0`](https://github.com/apache/spark/commit/6
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14745#discussion_r82559121
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVFileFormat.scala
---
@@ -57,28 +57,64 @@ class CSVFileFormat extends T
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15342
@sethah are you OK with this part? We can still talk about the k centroids
bit, either here or on the JIRA.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15371#discussion_r82559538
--- Diff: project/MimaExcludes.scala ---
@@ -55,7 +55,11 @@ object MimaExcludes {
ProblemFilters.exclude[ReversedMissingMethodProblem]("org.apache.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15414
**[Test build #66635 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66635/consoleFull)**
for PR 15414 at commit
[`6c2a8d0`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15414
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15414
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66635/
Test FAILed.
---
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14745
LGTM
---
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 fea
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15314
@sethah that's my fault, as I originally told @zhengruifeng that I
preferred to make these changes together. They sound closely related, but, if
you all think it should be separate after looking into
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/15414
test 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/15292
**[Test build #66636 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66636/consoleFull)**
for PR 15292 at commit
[`94389ef`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15414
**[Test build #66637 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66637/consoleFull)**
for PR 15414 at commit
[`6c2a8d0`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15292
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66636/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15292
**[Test build #66636 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66636/consoleFull)**
for PR 15292 at commit
[`94389ef`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15292
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15212
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66633/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15212
**[Test build #66633 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66633/consoleFull)**
for PR 15212 at commit
[`e141c68`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15212
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 philipphoffmann commented on the issue:
https://github.com/apache/spark/pull/14936
* fixed indent
* added documentation
any objections on the default timeout?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14963
Just so I'm doubly clear, this PR is ready to go, does something useful by
itself? that is it's not just like a change that only makes sense with others?
I'm OK to merge it if the Python folks think
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15414
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/15414
**[Test build #66637 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66637/consoleFull)**
for PR 15414 at commit
[`6c2a8d0`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15414
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66637/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15212
**[Test build #66634 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66634/consoleFull)**
for PR 15212 at commit
[`d05d7de`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15212
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/15212
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66634/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14936
**[Test build #66638 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66638/consoleFull)**
for PR 14936 at commit
[`6441f87`](https://github.com/apache/spark/commit/6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15319
**[Test build #66631 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66631/consoleFull)**
for PR 15319 at commit
[`1558d4c`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15319
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66631/
Test FAILed.
---
Github user yangw1234 commented on a diff in the pull request:
https://github.com/apache/spark/pull/15416#discussion_r82562070
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -2189,6 +2189,24 @@ class SQLQuerySuite extends QueryTest with
SharedSQL
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15319
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/14936
**[Test build #66638 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66638/consoleFull)**
for PR 14936 at commit
[`6441f87`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14936
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66638/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14936
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 HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15346
Hi @hvanhovell, it seems it fails to build now. I guess you might already
know but I am just leaving a comment here just in case to let you know.
```
[error]
/home/jenkins/workspac
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14936
**[Test build #66639 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66639/consoleFull)**
for PR 14936 at commit
[`ee10ad3`](https://github.com/apache/spark/commit/e
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/15346
@HyukjinKwon I'll fix that in a follow up. Thank you for telling!
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14936
**[Test build #66639 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66639/consoleFull)**
for PR 14936 at commit
[`ee10ad3`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14936
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66639/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14936
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 holdenk commented on the issue:
https://github.com/apache/spark/pull/14963
This PR does somehting useful by its self namely it uses virtual env /
`requirements.txt` to fix the versions of `pylint` and `pep8` and reenables
`pylint` checking of the code. There is a setup cos
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15295
**[Test build #66640 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66640/consoleFull)**
for PR 15295 at commit
[`fc78834`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15295
**[Test build #66640 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66640/consoleFull)**
for PR 15295 at commit
[`fc78834`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15295
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15295
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66640/
Test FAILed.
---
GitHub user jiangxb1987 opened a pull request:
https://github.com/apache/spark/pull/15417
[SPARK-17851][SQL][MINOR][TESTS] Update invalid test sql in
`ColumnPruningSuite`
## What changes were proposed in this pull request?
The `originalQuery` in `ColumnPruningSuite.test("Co
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/15417
cc @gatorsmile
---
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/15371
**[Test build #66632 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66632/consoleFull)**
for PR 15371 at commit
[`da2311a`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15371
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/15371
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66632/
Test PASSed.
---
Github user Stibbons commented on the issue:
https://github.com/apache/spark/pull/14963
Thanks both of you for your reviews.
It also fix the version of sphinx for documentation build during the
`lint-python` My point of view is that for **application**, all versions of
tools
GitHub user jiangxb1987 opened a pull request:
https://github.com/apache/spark/pull/15418
[MINOR][SQL][TESTS] Remove unused function in `SparkSqlParserSuite`
## What changes were proposed in this pull request?
The function `SparkSqlParserSuite.createTempViewUsing` is not use
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15418
Oops, yeah looks like this is a hotfix. Weird, because
https://github.com/apache/spark/pull/14897 passed very recently. CC @cloud-fan
? not exactly sure if this is the cause.
```
[error]
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15418
OK to test
---
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 t
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/15389#discussion_r82570191
--- Diff: python/pyspark/rdd.py ---
@@ -2029,7 +2028,11 @@ def coalesce(self, numPartitions, shuffle=False):
>>> sc.parallelize([1, 2, 3, 4, 5], 3
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15418
it's due to a merge conflict(2 PRs do conflict but their code doesn't)
I'll merge this PR as soon as it pass the jenkins. @jiangxb1987 can you
update the title to remove [minor] and add [h
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15418
ok to test
---
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 t
GitHub user a-roberts opened a pull request:
https://github.com/apache/spark/pull/15419
[SPARK-17828] [DOCS] Remove unused generate-changelist.py
## What changes were proposed in this pull request?
We can remove this file based on discussion at
https://issues.apache.org/jira/bro
1 - 100 of 799 matches
Mail list logo