Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22153
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95008/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22153
**[Test build #95008 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95008/testReport)**
for PR 22153 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21961
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21610
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22118
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21624
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18424
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21794
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16411
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22159
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21870
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19528
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21940
---
-
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/22157
**[Test build #4283 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4283/testReport)**
for PR 22157 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/22152#discussion_r211599957
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JsonInferSchema.scala
---
@@ -69,10 +70,17 @@ private[sql] object JsonInferSchema
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22135
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/22135
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/22135
**[Test build #95017 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95017/testReport)**
for PR 22135 at commit
Github user vincent-grosbois commented on the issue:
https://github.com/apache/spark/pull/22024
Hi,
sorry I haven't done any benchmark on this.
But this solved an Out Of Memory issue for:
- before this fix with Spark 2.3 we had OOM issues in specific jobs when
the
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22152#discussion_r211584402
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -116,16 +129,24 @@ object SQLConf {
if (TaskContext.get
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22152#discussion_r211585144
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JsonInferSchema.scala
---
@@ -69,10 +70,17 @@ private[sql] object
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/22164
@vanzin @tgravescs would you please help to review, thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22024
@vincent-grosbois I don't look into this change yet. Do you have reply for
@dbtsai's comment
https://github.com/apache/spark/pull/22024#issuecomment-412659532?
---
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22024
retest this please.
---
-
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/22163
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/22163
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/22163
**[Test build #95016 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95016/testReport)**
for PR 22163 at commit
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/22163
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user sujithjay commented on a diff in the pull request:
https://github.com/apache/spark/pull/22168#discussion_r211579406
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeJoinExec.scala
---
@@ -1099,7 +,7 @@ private class
Github user vincent-grosbois commented on the issue:
https://github.com/apache/spark/pull/22024
Hi @viirya , any news on this? do you need me to do something?
---
-
To unsubscribe, e-mail:
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/22168#discussion_r211577003
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/SortMergeJoinExec.scala
---
@@ -1099,7 +,7 @@ private class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22154
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/22154
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95004/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22154
**[Test build #95004 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95004/testReport)**
for PR 22154 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22158
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95000/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22158
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/22158
**[Test build #95000 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95000/testReport)**
for PR 22158 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21733
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/19691
@DazhuangSu Can you resolve the conflict?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21733
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95003/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21733
**[Test build #95003 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95003/testReport)**
for PR 21733 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22154
**[Test build #95015 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95015/testReport)**
for PR 22154 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22154
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/22154
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22009
@cloud-fan, are there more things to be done here than rebasing?
---
-
To unsubscribe, e-mail:
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/22154
@rednaxelafx Thanks for your checks ;) addressed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22112
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/22112
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95005/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22112
**[Test build #95005 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95005/testReport)**
for PR 22112 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22154
**[Test build #95014 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95014/testReport)**
for PR 22154 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22170
**[Test build #95013 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95013/testReport)**
for PR 22170 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22170
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22154
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/22154
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/22170
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22170
cc. @tdas
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
GitHub user HeartSaVioR opened a pull request:
https://github.com/apache/spark/pull/22170
[MINOR] Add .crc files to .gitignore
## What changes were proposed in this pull request?
Add .crc files to .gitignore so that we don't add .crc files in state
checkpoint during tests
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22133
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/22133
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95001/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22133
**[Test build #95001 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95001/testReport)**
for PR 22133 at commit
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/22154#discussion_r211565872
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CodeGeneratorWithInterpretedFallbackSuite.scala
---
@@ -40,4 +55,13 @@
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/22154#discussion_r211565120
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CodeGeneratorWithInterpretedFallbackSuite.scala
---
@@ -40,4 +55,13 @@
Github user eyalfa commented on a diff in the pull request:
https://github.com/apache/spark/pull/17400#discussion_r211565064
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/EnsureRequirements.scala
---
@@ -321,6 +321,58 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22166
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95002/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22166
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/22166
**[Test build #95002 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95002/testReport)**
for PR 22166 at commit
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/22133#discussion_r211561656
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -626,6 +626,7 @@ object DataSource extends
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17400#discussion_r211561522
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/EnsureRequirements.scala
---
@@ -321,6 +321,58 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21469
**[Test build #95012 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95012/testReport)**
for PR 21469 at commit
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21469
@tdas Thanks for the feedback! Updated the PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
GitHub user hn5092 opened a pull request:
https://github.com/apache/spark/pull/22169
Kyspark 2.1.1 r3
## What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
## How was this patch tested?
(Please explain how this
Github user hn5092 closed the pull request at:
https://github.com/apache/spark/pull/22169
---
-
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/21912
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/21912
**[Test build #95011 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95011/testReport)**
for PR 21912 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21912
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/22124
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/22124
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95006/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17400
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/17400
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95009/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17400
**[Test build #95009 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95009/testReport)**
for PR 17400 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22124
**[Test build #95006 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95006/testReport)**
for PR 22124 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20838
**[Test build #95010 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95010/testReport)**
for PR 20838 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17400
**[Test build #95009 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95009/testReport)**
for PR 17400 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17400
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/17400
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cclauss closed the pull request at:
https://github.com/apache/spark/pull/22128
---
-
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/22153
**[Test build #95008 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95008/testReport)**
for PR 22153 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22153
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/22153
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/22153
retest this please
---
-
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/22161#discussion_r211541767
--- Diff: R/pkg/tests/fulltests/test_sparkSQL.R ---
@@ -3640,17 +3640,18 @@ test_that("catalog APIs, listTables, listColumns,
listFunctions", {
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22161
Seems okay to me.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20838
+1 for ^. We can credit to multiple persons now anyway.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22128
I think that comment suggested to add `flake8` in the testing framework.
Let's better fix these in the same PR and leave this closed.
---
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22133#discussion_r211537581
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -626,6 +626,7 @@ object DataSource extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22165
**[Test build #95007 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95007/testReport)**
for PR 22165 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22165
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/22165
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/22165
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user sujithjay commented on the issue:
https://github.com/apache/spark/pull/22168
Hi @tejasapatil, @viirya, @hvanhovell, & @kiszk, can you please review
this pull request?
---
-
To unsubscribe, e-mail:
601 - 700 of 795 matches
Mail list logo
