Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22382
Since it's merged, could you close this PR, @bersprockets ?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13893
Ping, @jiangxb1987 .
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: r
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22211
Hi, @henryr . Since this is merged, could you close this PR?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22337
**[Test build #95917 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95917/testReport)**
for PR 22337 at commit
[`2d9e34a`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22389
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/22192
**[Test build #95915 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95915/testReport)**
for PR 22192 at commit
[`447c5e5`](https://github.com/apache/spark/commit/4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22389
**[Test build #95922 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95922/testReport)**
for PR 22389 at commit
[`9a04d87`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22376
**[Test build #95913 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95913/testReport)**
for PR 22376 at commit
[`4a0cffb`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21596
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/21596
**[Test build #95919 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95919/testReport)**
for PR 21596 at commit
[`bb8b661`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22389
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95922/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/7
**[Test build #95918 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95918/testReport)**
for PR 7 at commit
[`b12ee88`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22387
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/22385
**[Test build #95914 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95914/testReport)**
for PR 22385 at commit
[`daf76ed`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21596
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95919/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22387
**[Test build #95921 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95921/testReport)**
for PR 22387 at commit
[`a7b857c`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22213
**[Test build #95920 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95920/testReport)**
for PR 22213 at commit
[`76635ce`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22337
Merged build finished. Test FAILed.
---
-
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/22213
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95920/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22192
Merged build finished. Test FAILed.
---
-
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/22385
Merged build finished. Test FAILed.
---
-
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/22213
Merged build finished. Test FAILed.
---
-
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/22192
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95915/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/7
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95918/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22387
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95921/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22385
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95914/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22337
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95917/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/7
Merged build finished. Test FAILed.
---
-
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/22376
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95913/
Test FAILed.
---
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/22390
[SPARK-25402][SQL] Null handling in BooleanSimplification
## What changes were proposed in this pull request?
This PR is to fix the null handling in BooleanSimplification. In the rule
Boolea
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22376
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22390
cc @cloud-fan @adrian-ionescu
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-m
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22390
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/3002/
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22389
retest this please
---
-
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/22390
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/22390
**[Test build #95923 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95923/testReport)**
for PR 22390 at commit
[`cf863bb`](https://github.com/apache/spark/commit/cf
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22389
**[Test build #95924 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95924/testReport)**
for PR 22389 at commit
[`9a04d87`](https://github.com/apache/spark/commit/9a
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/22357#discussion_r216565635
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetSchemaPruning.scala
---
@@ -110,7 +110,17 @@ private[sql] object
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22387
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/20504
I will close it now.
---
-
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/22387
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/22387
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/3003/
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22388
yes
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apac
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22387
**[Test build #95925 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95925/testReport)**
for PR 22387 at commit
[`a7b857c`](https://github.com/apache/spark/commit/a7
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/22374#discussion_r216569630
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala
---
@@ -1700,4 +1700,13 @@ class CSVSuite extends QueryTest
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22364#discussion_r216572034
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/AttributeSet.scala
---
@@ -39,10 +41,15 @@ object AttributeSet {
Github user wangyum closed the pull request at:
https://github.com/apache/spark/pull/20504
---
-
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/22341#discussion_r216574255
--- Diff:
core/src/main/scala/org/apache/spark/status/AppStatusListener.scala ---
@@ -646,7 +647,47 @@ private[spark] class AppStatusListener(
}
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22341
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/3004/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22341
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22341
@vanzin Thanks for the good review! I've updated this to address them all.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22341
**[Test build #95926 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95926/testReport)**
for PR 22341 at commit
[`7c76790`](https://github.com/apache/spark/commit/7c
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22390#discussion_r216575397
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala
---
@@ -263,10 +263,12 @@ object BooleanSimplification ex
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/22390#discussion_r216575336
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/BooleanSimplificationSuite.scala
---
@@ -48,6 +48,14 @@ class BooleanSi
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/22390#discussion_r216574955
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/BooleanSimplificationSuite.scala
---
@@ -48,6 +48,14 @@ class BooleanSi
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/22390#discussion_r216576135
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala
---
@@ -263,10 +263,12 @@ object BooleanSimplificatio
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22390
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95923/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22390
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/22390
**[Test build #95923 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95923/testReport)**
for PR 22390 at commit
[`cf863bb`](https://github.com/apache/spark/commit/c
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22343
To clarify: this is just a workaround when we hit a problematic(having
case-insensitive duplicated filed names in the parquet file) hive parquet
tables and we want to read it with the native parqu
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22371
@ConeyLiu we may have an executor lost and then come back, and may have 2
same tasks running on the same executor.
---
-
To un
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22343
Thank you for the pointer, @seancxmao . And thank you for clarification,
@cloud-fan .
It looks like we are re-creating correctness issue somewhat in this PR when
`caseSensitive=true`.
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22371
My opinion is, it's not worth to spend time on it. The lock is not likely
to be a bottleneck and it's better to keep it simple even it's sub-optimal.
---
GitHub user mgaido91 opened a pull request:
https://github.com/apache/spark/pull/22391
[SPARK-25371][SQL][BACKPORT-2.3] struct() should allow being called with 0
args
## What changes were proposed in this pull request?
SPARK-21281 introduced a check for the inputs of `Creat
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22391
cc @cloud-fan @dongjoon-hyun
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22378#discussion_r216580577
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/InsertSuite.scala ---
@@ -750,4 +751,27 @@ class InsertSuite extends QueryTest with
TestHiveS
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22378
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22391
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/3005/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22391
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/22369
As the comment in
https://github.com/apache/spark/pull/22140#issuecomment-419997180, I think this
doc change is no more needed, I just close this, thanks @BryanCutler and
@HyukjinKwon !
---
Github user xuanyuanking closed the pull request at:
https://github.com/apache/spark/pull/22369
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.o
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22391
**[Test build #95927 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95927/testReport)**
for PR 22391 at commit
[`66b6bd5`](https://github.com/apache/spark/commit/66
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22367
Usually we merge into master and backport to other branches when it's
needed.
https://spark.apache.org/contributing.html
> 5. Open a pull request against the master branch of ap
Github user xuanyuanking commented on a diff in the pull request:
https://github.com/apache/spark/pull/22326#discussion_r216583509
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/FilterPushdownSuite.scala
---
@@ -1153,12 +1154,35 @@ class FilterPushdo
Github user xuanyuanking commented on a diff in the pull request:
https://github.com/apache/spark/pull/22326#discussion_r216585000
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/FilterPushdownSuite.scala
---
@@ -1153,12 +1154,35 @@ class FilterPushdo
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22357
Can anyone point me out if there are non addressed comments or problems
here? Looks pretty good to me. I think this is rather a bandaid, small and safe
fix to get into branch-2.4.
---
---
GitHub user ssaavedra opened a pull request:
https://github.com/apache/spark/pull/22392
[SPARK-23200] Reset Kubernetes-specific config on Checkpoint restore
Several configuration parameters related to Kubernetes need to be
reset, as they are changed with each invokation of spark-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22392
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user ConeyLiu closed the pull request at:
https://github.com/apache/spark/pull/22371
---
-
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/22385
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/22371
OK, thanks everyone for the help. Close it
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional com
Github user xuanyuanking commented on a diff in the pull request:
https://github.com/apache/spark/pull/21618#discussion_r216587584
--- Diff:
core/src/test/scala/org/apache/spark/deploy/SparkHadoopUtilSuite.scala ---
@@ -77,6 +80,51 @@ class SparkHadoopUtilSuite extends SparkFunSuit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22392
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/22392
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/22385
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/3006/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22385
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/22385
**[Test build #95928 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95928/testReport)**
for PR 22385 at commit
[`daf76ed`](https://github.com/apache/spark/commit/da
Github user xuanyuanking commented on a diff in the pull request:
https://github.com/apache/spark/pull/21618#discussion_r216588353
--- Diff: core/src/main/java/org/apache/hadoop/fs/SparkGlobber.java ---
@@ -0,0 +1,293 @@
+/**
+ * Licensed to the Apache Software Foundation (A
Github user adrian-ionescu commented on a diff in the pull request:
https://github.com/apache/spark/pull/22390#discussion_r216587101
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala
---
@@ -263,10 +263,12 @@ object BooleanSimplificati
Github user adrian-ionescu commented on a diff in the pull request:
https://github.com/apache/spark/pull/22390#discussion_r216587787
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala
---
@@ -263,10 +263,12 @@ object BooleanSimplificati
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21080
@dzanozin the code you reported clearly misses this patch. Since I do see
the commit on v2.3.1, I am a bit puzzled about the issue you reported. Please
provide a reproducer so we can check whether
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21596
**[Test build #95929 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95929/testReport)**
for PR 21596 at commit
[`98d459f`](https://github.com/apache/spark/commit/98
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22326#discussion_r216592676
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/FilterPushdownSuite.scala
---
@@ -1153,12 +1154,35 @@ class FilterPushdownSu
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20999#discussion_r216593044
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -293,6 +293,28 @@ class AstBuilder(conf: SQLConf) exten
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20999
cc @cloud-fan @dongjoon-hyun too, in case have time to take a look and
review this, thanks.
---
-
To unsubscribe, e-mail: revie
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/22374#discussion_r216593835
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVDataSource.scala
---
@@ -243,17 +243,21 @@ object TextInputCSVDat
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22358
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/3007/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22358
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/22358
**[Test build #95930 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95930/testReport)**
for PR 22358 at commit
[`dd86d3f`](https://github.com/apache/spark/commit/dd
Github user Ngone51 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18492#discussion_r216597619
--- Diff:
core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala ---
@@ -373,8 +373,14 @@ private[spark] class ExecutorAllocationManager(
1 - 100 of 585 matches
Mail list logo
