Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/22944#discussion_r232928066
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetSuite.scala
---
@@ -1556,6 +1556,20 @@ class DatasetSuite extends QueryTest with
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/22944#discussion_r232926151
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetSuite.scala
---
@@ -1556,6 +1556,20 @@ class DatasetSuite extends QueryTest with
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/23009#discussion_r232921575
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -318,7 +318,7 @@ private[spark] class SparkSubmit extends Logging {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23020
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23020
**[Test build #98758 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98758/testReport)**
for PR 23020 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23020
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user srowen opened a pull request:
https://github.com/apache/spark/pull/23020
[MINOR][BUILD] Remove *.crc from .gitignore
## What changes were proposed in this pull request?
Remove *.crc from .gitignore as there are actual .crc files in the test
source dirs and IJ
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22989
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22989
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98749/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22989
**[Test build #98749 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98749/testReport)**
for PR 22989 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22974
**[Test build #98757 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98757/testReport)**
for PR 22974 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22974
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22974
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22989#discussion_r23291
--- Diff: dev/checkstyle-suppressions.xml ---
@@ -46,4 +46,12 @@
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22989#discussion_r232917995
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/feature/VectorIndexerSuite.scala ---
@@ -283,7 +283,9 @@ class VectorIndexerSuite extends MLTest with
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22967#discussion_r232916947
--- Diff: pom.xml ---
@@ -2718,7 +2710,6 @@
*:*_2.11
-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23014
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98748/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23014
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23014
**[Test build #98748 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98748/testReport)**
for PR 23014 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22967
**[Test build #98756 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98756/testReport)**
for PR 22967 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22967
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22967
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22974
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98747/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22974
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22974
**[Test build #98747 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98747/testReport)**
for PR 22974 at commit
Github user LiehuoChen commented on the issue:
https://github.com/apache/spark/pull/21588
Hi HyukjinKwon,
Thanks for all the works to try to make the Jenkin test pass.
I patched this PR to spark 2.4, and anything works fine but failed in
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22977
---
-
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/22954
**[Test build #98755 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98755/testReport)**
for PR 22954 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22977
since this PR only touches mima, and the jenkins already passed the mima
check, I'm going to merge it to master, thanks!
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22954
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22954
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/22977
LGTM. Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22518
BTW can you include a simple benchmark to show this problem? e.g. just run
a query in spark-shell, and post the result before and after this PR.
---
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22518
I'd like to merge this simple PR first, to address the performance problem
(unnecessary subquery execution).
Let's create a new ticket for subquery filter pushing to data source, and
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22518#discussion_r232906707
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/PruneFileSourcePartitions.scala
---
@@ -47,7 +47,8 @@ private[sql] object
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22518#discussion_r232906743
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SubquerySuite.scala
---
@@ -1268,4 +1269,16 @@ class SubquerySuite extends QueryTest with
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22518#discussion_r232906652
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileSourceStrategy.scala
---
@@ -155,15 +155,14 @@ object
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22961#discussion_r232906123
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/ShuffleExchangeExec.scala
---
@@ -214,13 +214,22 @@ object ShuffleExchangeExec
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22944#discussion_r232905784
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetSuite.scala
---
@@ -1556,6 +1556,20 @@ class DatasetSuite extends QueryTest with
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/23002
---
-
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 the issue:
https://github.com/apache/spark/pull/23002
thanks, merging to master!
---
-
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/23014
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23014
**[Test build #98754 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98754/testReport)**
for PR 23014 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23014
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22721
cc @jiangxb1987 Could you take a look at this?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22977
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22977
**[Test build #98753 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98753/testReport)**
for PR 22977 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22977
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21465
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21465
**[Test build #98751 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98751/testReport)**
for PR 21465 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21465
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98751/
Test PASSed.
---
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22954#discussion_r232895848
--- Diff: R/pkg/R/SQLContext.R ---
@@ -172,36 +257,72 @@ getDefaultSqlSource <- function() {
createDataFrame <- function(data, schema = NULL,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22944
**[Test build #98752 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98752/testReport)**
for PR 22944 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22944
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22944
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21465
**[Test build #98751 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98751/testReport)**
for PR 21465 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22977
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22977
**[Test build #98750 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98750/testReport)**
for PR 22977 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22977
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98750/
Test FAILed.
---
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/23006
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/22944#discussion_r232894304
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetSuite.scala
---
@@ -1556,6 +1556,20 @@ class DatasetSuite extends QueryTest with
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23006
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/23014#discussion_r232893546
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/WritableColumnVector.java
---
@@ -101,10 +101,11 @@ private void
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22977
**[Test build #98750 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98750/testReport)**
for PR 22977 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22977
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22977
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/22989
@srowen Great thanks for your guidance, address all your suggestion in
ff234d3 and update the record table in
https://github.com/apache/spark/pull/22989#issuecomment-437939830.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22989
**[Test build #98749 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98749/testReport)**
for PR 22989 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22989
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22989
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/23014
Yes. The `filePartitions` are the same as the bucket number when
`BucketedRead`:
Github user mu5358271 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22961#discussion_r232888324
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/ShuffleExchangeExec.scala
---
@@ -214,13 +214,22 @@ object ShuffleExchangeExec
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22977
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22977
**[Test build #98746 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98746/testReport)**
for PR 22977 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22977
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98746/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23014
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23014
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23014
**[Test build #98748 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98748/testReport)**
for PR 23014 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23014
> The reason is that each bucket file is too big
Can you elaborate please? Is it because we don't chunk each file into
multiple splits when we read bucketed table?
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22974
**[Test build #98747 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98747/testReport)**
for PR 22974 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22974
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22974
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22977
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22977
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22977
**[Test build #98746 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98746/testReport)**
for PR 22977 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22977#discussion_r232886260
--- Diff: project/MimaExcludes.scala ---
@@ -164,7 +212,50 @@ object MimaExcludes {
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/23014#discussion_r232885260
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/WritableColumnVector.java
---
@@ -101,7 +101,8 @@ private void
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/23007
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21688
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21688
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98745/
Test PASSed.
---
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/23007
merged to master/2.4
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21688
**[Test build #98745 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98745/testReport)**
for PR 21688 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/22866
thx, but DO NOT MERGE - there's some nasty bug I'm still investigating..
---
-
To unsubscribe, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23018
Looks fine to me. adding @cloud-fan and @hvanhovell
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/23018#discussion_r232883084
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala
---
@@ -469,7 +471,21 @@ abstract class TreeNode[BaseType <:
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/23012#discussion_r232881732
--- Diff: R/pkg/R/sparkR.R ---
@@ -283,6 +283,10 @@ sparkR.session <- function(
enableHiveSupport = TRUE,
...) {
+ if
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/23012#discussion_r232882419
--- Diff: docs/index.md ---
@@ -31,7 +31,8 @@ Spark runs on both Windows and UNIX-like systems (e.g.
Linux, Mac OS). It's easy
locally on one
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/23012#discussion_r232882178
--- Diff: docs/index.md ---
@@ -31,7 +31,8 @@ Spark runs on both Windows and UNIX-like systems (e.g.
Linux, Mac OS). It's easy
locally on one
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/23012#discussion_r232881594
--- Diff: R/WINDOWS.md ---
@@ -3,7 +3,7 @@
To build SparkR on Windows, the following steps are required
1. Install R (>= 3.1) and
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23012
In this way, we could postpone R upgrade after Spark 3.0.0 release in
Jenkins, and could still test the deprecated R version 3.1.
---
1 - 100 of 403 matches
Mail list logo