Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21669
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92471/
Test FAILed.
---
Github user Fokko commented on the issue:
https://github.com/apache/spark/pull/21596
It took some time, but here it is, current master:
```
MacBook-Pro-van-Fokko:dist fokkodriesprong$ cat /tmp/spark-master.txt
Preparing data for benchmarking ...
Running benchmark: JSON
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21669
**[Test build #92471 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92471/testReport)**
for PR 21669 at commit
Github user dedunumax commented on a diff in the pull request:
https://github.com/apache/spark/pull/21120#discussion_r199135313
--- Diff: mllib/src/main/scala/org/apache/spark/ml/stat/Summarizer.scala ---
@@ -562,6 +573,23 @@ private[ml] object SummaryBuilderImpl extends Logging {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21659
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92462/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21659
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/21659
**[Test build #92462 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92462/testReport)**
for PR 21659 at commit
Github user phatak-dev commented on the issue:
https://github.com/apache/spark/pull/20791
This fix is not showing up in 2.3.1 downloadable.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21668
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/21667
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92465/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20629
**[Test build #92477 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92477/testReport)**
for PR 20629 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20629
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/587/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21667
**[Test build #92465 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92465/testReport)**
for PR 21667 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21668
**[Test build #92460 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92460/testReport)**
for PR 21668 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21667
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/20629
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/21668
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92460/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20629
**[Test build #92476 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92476/testReport)**
for PR 20629 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21611
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92464/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21611
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/21623
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/21623
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92463/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21611
**[Test build #92464 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92464/testReport)**
for PR 21611 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21623
**[Test build #92463 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92463/testReport)**
for PR 21623 at commit
Github user caneGuy commented on the issue:
https://github.com/apache/spark/pull/21664
cc @jiangxb1987 @jerryshao Could you help review this?Thanks
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20629
**[Test build #92475 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92475/testReport)**
for PR 20629 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20629
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92475/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20629
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gaborgsomogyi commented on the issue:
https://github.com/apache/spark/pull/21455
Sounds weird. The topic names are available when the streaming app
submitted. Why this can't be reused? Second option is reflection as jerryshao
suggested.
---
Github user mahmoudmahdi24 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21120#discussion_r199119223
--- Diff: mllib/src/main/scala/org/apache/spark/ml/stat/Summarizer.scala ---
@@ -562,6 +573,23 @@ private[ml] object SummaryBuilderImpl extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20345
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92466/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21623
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/586/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20345
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/20345
**[Test build #92466 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92466/testReport)**
for PR 20345 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21623
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/20629
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/585/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20629
**[Test build #92475 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92475/testReport)**
for PR 20629 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21623
**[Test build #92474 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92474/testReport)**
for PR 21623 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20629
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user wangyum commented on a diff in the pull request:
https://github.com/apache/spark/pull/21623#discussion_r199116993
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFilterSuite.scala
---
@@ -660,6 +688,62 @@ class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21403#discussion_r199116440
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala
---
@@ -161,33 +161,38 @@ case class Not(child:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21619#discussion_r199116305
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/javaCode.scala
---
@@ -148,14 +149,17 @@ trait Block extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21583
Test FAILed.
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/584/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21583
Kubernetes integration test status failure
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/584/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21583
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/21583
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/584/
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21583
**[Test build #92473 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92473/testReport)**
for PR 21583 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21619#discussion_r199105735
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/javaCode.scala
---
@@ -148,14 +149,17 @@ trait Block extends
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21659#discussion_r199104696
--- Diff: python/docs/Makefile ---
@@ -1,19 +1,35 @@
# Makefile for Sphinx documentation
#
+ifndef SPHINXPYTHON
+SPHINXBUILD ?=
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21109
**[Test build #92472 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92472/testReport)**
for PR 21109 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21659
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92469/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21659
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/21659
**[Test build #92469 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92469/testReport)**
for PR 21659 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21659
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92468/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21659
**[Test build #92468 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92468/testReport)**
for PR 21659 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21659
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21659#discussion_r199099575
--- Diff: python/docs/Makefile ---
@@ -1,19 +1,35 @@
# Makefile for Sphinx documentation
#
+ifndef SPHINXPYTHON
+SPHINXBUILD ?=
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21669
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/583/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21669
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/583/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21669
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21659#discussion_r199098475
--- Diff: python/docs/Makefile ---
@@ -1,19 +1,35 @@
# Makefile for Sphinx documentation
#
+ifndef SPHINXPYTHON
+SPHINXBUILD ?=
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21669
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/583/
---
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21659#discussion_r199096185
--- Diff: python/docs/Makefile ---
@@ -1,19 +1,35 @@
# Makefile for Sphinx documentation
#
+ifndef SPHINXPYTHON
+SPHINXBUILD ?=
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21659#discussion_r199096928
--- Diff: python/docs/Makefile ---
@@ -1,19 +1,35 @@
# Makefile for Sphinx documentation
#
+ifndef SPHINXPYTHON
+SPHINXBUILD ?=
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21667#discussion_r199096057
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileFormat.scala
---
@@ -152,6 +152,16 @@ trait FileFormat {
}
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21667#discussion_r199095666
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileFormat.scala
---
@@ -152,6 +152,16 @@ trait FileFormat {
}
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21659
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/21669
**[Test build #92471 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92471/testReport)**
for PR 21669 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21659
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/582/
GitHub user ifilonenko opened a pull request:
https://github.com/apache/spark/pull/21669
[SPARK-23257][K8S][WIP] Kerberos Support for Spark on K8S
## What changes were proposed in this pull request?
This is the work on setting up Secure HDFS interaction with Spark-on-K8S.
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21667#discussion_r199094171
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/orc/HiveOrcSourceSuite.scala
---
@@ -156,28 +156,6 @@ class HiveOrcSourceSuite extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21659
**[Test build #92469 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92469/testReport)**
for PR 21659 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21321
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/21321
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/581/
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21659
Now, it keeps the previous behaviour and use the Python iff `SPHINXPYTHON`
is set.
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21321
**[Test build #92470 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92470/testReport)**
for PR 21321 at commit
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21321
Thanks for your kind and nice review @holdenk, I addressed the places I
missed. Thank you!
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21659
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/580/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21659
**[Test build #92468 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92468/testReport)**
for PR 21659 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21659
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21321#discussion_r199091397
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/FeatureHasher.scala ---
@@ -208,7 +208,8 @@ class FeatureHasher(@Since("2.3.0") override val
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20629
Thanks @holdenk @sethah, then I'll update this PR accordingly. Thanks.
---
-
To unsubscribe, e-mail:
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21403#discussion_r199087446
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala
---
@@ -161,33 +161,38 @@ case class Not(child:
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21403#discussion_r199086016
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala
---
@@ -161,33 +161,38 @@ case class Not(child:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21623#discussion_r199081727
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFilterSuite.scala
---
@@ -660,6 +688,62 @@ class
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/21667#discussion_r199081671
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/orc/HiveOrcSourceSuite.scala
---
@@ -156,28 +156,6 @@ class HiveOrcSourceSuite extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21109
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92467/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21109
**[Test build #92467 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92467/testReport)**
for PR 21109 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21109
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21623#discussion_r199080962
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFilterSuite.scala
---
@@ -140,6 +142,32 @@ class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21109
**[Test build #92467 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92467/testReport)**
for PR 21109 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21403#discussion_r199080792
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala
---
@@ -161,33 +161,38 @@ case class Not(child:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21666
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/21666
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92461/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21666
**[Test build #92461 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92461/testReport)**
for PR 21666 at commit
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21403#discussion_r199080189
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala
---
@@ -161,33 +161,38 @@ case class Not(child:
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/21667#discussion_r199079744
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileFormat.scala
---
@@ -152,6 +152,16 @@ trait FileFormat {
}
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21403#discussion_r199079546
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala
---
@@ -161,33 +161,38 @@ case class Not(child:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21623#discussion_r199078944
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFilters.scala
---
@@ -270,6 +273,36 @@ private[parquet]
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21623#discussion_r199078056
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -378,6 +378,14 @@ object SQLConf {
.booleanConf
201 - 300 of 388 matches
Mail list logo
