Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21529
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/21529
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/3983/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21282
**[Test build #91774 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91774/testReport)**
for PR 21282 at commit
[`10ace84`](https://github.com/apache/spark/commit/10
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21539
thanks, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21539
---
-
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/21282
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20636#discussion_r195107427
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/codegen/BufferHolder.java
---
@@ -61,12 +61,15 @@
* Grows the buff
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20636#discussion_r195108390
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/codegen/BufferHolder.java
---
@@ -61,12 +61,15 @@
* Grows the buff
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21260
**[Test build #91775 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91775/testReport)**
for PR 21260 at commit
[`f714b8e`](https://github.com/apache/spark/commit/f7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21441
**[Test build #91776 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91776/testReport)**
for PR 21441 at commit
[`da92e01`](https://github.com/apache/spark/commit/da
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20636#discussion_r195110654
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/codegen/BufferHolder.java
---
@@ -61,12 +61,15 @@
* Grows the buff
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20636#discussion_r195111027
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/codegen/BufferHolder.java
---
@@ -61,12 +61,15 @@
* Grows the buff
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21529#discussion_r195099853
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/PlannerSuite.scala ---
@@ -679,6 +679,23 @@ class PlannerSuite extends SharedSQLContext {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21529#discussion_r195101023
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/JoinSuite.scala ---
@@ -882,4 +882,15 @@ class JoinSuite extends QueryTest with
SharedSQLContext {
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21529
LGTM except some minor comments about test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional co
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21529#discussion_r195100665
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/JoinSuite.scala ---
@@ -882,4 +882,15 @@ class JoinSuite extends QueryTest with
SharedSQLContext {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21260
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/21260
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/3984/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21260
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3836/
---
--
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21441
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/94/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21441
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/21441
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/3985/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21441
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/21260
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3836/
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21260
**[Test build #91775 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91775/testReport)**
for PR 21260 at commit
[`f714b8e`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21260
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91775/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21260
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21441
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21441
**[Test build #91777 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91777/testReport)**
for PR 21441 at commit
[`da92e01`](https://github.com/apache/spark/commit/da
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21441
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21549#discussion_r195118451
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/xml/UDFXPathUtilSuite.scala
---
@@ -77,6 +77,23 @@ class UDFXPathUtilSuit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21441
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/3986/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21441
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/21441
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/95/
Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/21553
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user xuanyuanking commented on a diff in the pull request:
https://github.com/apache/spark/pull/17702#discussion_r195124437
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala
---
@@ -252,6 +252,18 @@ class SparkHadoopUtil extends Logging {
i
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21553
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/21553
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/96/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21553
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/21553
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/3987/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21553
**[Test build #91778 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91778/testReport)**
for PR 21553 at commit
[`8d33af7`](https://github.com/apache/spark/commit/8d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21547
**[Test build #91768 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91768/testReport)**
for PR 21547 at commit
[`1170a45`](https://github.com/apache/spark/commit/1
Github user xuanyuanking closed the pull request at:
https://github.com/apache/spark/pull/17702
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.o
Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/17702
```
This approach only works if the first level glob pattern matches a lot of
directories.
```
Yep, actually in our internal usage, we leave the problem to user and they
should use
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21547
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91768/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21547
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/21441
**[Test build #91770 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91770/testReport)**
for PR 21441 at commit
[`a57dd09`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21441
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91770/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21441
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/21068
**[Test build #91779 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91779/testReport)**
for PR 21068 at commit
[`7fce4ee`](https://github.com/apache/spark/commit/7f
Github user xuanyuanking commented on a diff in the pull request:
https://github.com/apache/spark/pull/21533#discussion_r195132870
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -1517,9 +1517,12 @@ class SparkContext(config: SparkConf) extends
Logging {
Github user xuanyuanking commented on a diff in the pull request:
https://github.com/apache/spark/pull/21533#discussion_r195133036
--- Diff: core/src/test/scala/org/apache/spark/SparkContextSuite.scala ---
@@ -116,49 +116,52 @@ class SparkContextSuite extends SparkFunSuite with
Loc
Github user xuanyuanking commented on a diff in the pull request:
https://github.com/apache/spark/pull/21533#discussion_r195133122
--- Diff: core/src/test/scala/org/apache/spark/SparkContextSuite.scala ---
@@ -116,49 +116,52 @@ class SparkContextSuite extends SparkFunSuite with
Loc
Github user xuanyuanking commented on a diff in the pull request:
https://github.com/apache/spark/pull/21533#discussion_r195134460
--- Diff: core/src/test/scala/org/apache/spark/SparkContextSuite.scala ---
@@ -116,49 +116,52 @@ class SparkContextSuite extends SparkFunSuite with
Loc
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21533
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/21533
**[Test build #91780 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91780/testReport)**
for PR 21533 at commit
[`797cefe`](https://github.com/apache/spark/commit/79
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21533
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/97/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21428
**[Test build #91781 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91781/testReport)**
for PR 21428 at commit
[`cff37c4`](https://github.com/apache/spark/commit/cf
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21533
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/21533
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/3988/
Tes
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/21503#discussion_r195138932
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Strategy.scala
---
@@ -17,15 +17,56 @@
package org.ap
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21503
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/3989/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21503
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/21503
Retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/16722
Hi @sethah is this something you're still working on or would it be an OK
PR for someone else to take over and maybe you could do the review side
together?
---
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21503
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/21503
**[Test build #91782 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91782/testReport)**
for PR 21503 at commit
[`c8517e1`](https://github.com/apache/spark/commit/c8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21503
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/98/
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21533#discussion_r195143756
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -1517,9 +1517,12 @@ class SparkContext(config: SparkConf) extends
Logging {
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21450#discussion_r19515
--- Diff: launcher/src/main/java/org/apache/spark/launcher/Main.java ---
@@ -101,6 +99,22 @@ public static void main(String[] argsArray) throws
Exception {
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21247#discussion_r195145029
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JSONOptions.scala
---
@@ -138,3 +121,40 @@ private[sql] class JSONOptions(
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/21247
Jenkins ok to test.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21556
**[Test build #91769 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91769/testReport)**
for PR 21556 at commit
[`9832661`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21556
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91769/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21556
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/21247
**[Test build #91783 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91783/testReport)**
for PR 21247 at commit
[`2285652`](https://github.com/apache/spark/commit/22
Github user susanxhuynh commented on the issue:
https://github.com/apache/spark/pull/20641
@skonto Thanks for testing it. Tests results look good.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Fo
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21247#discussion_r195151361
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JSONOptions.scala
---
@@ -94,32 +94,15 @@ private[sql] class JSONOptions(
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20629#discussion_r195153337
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/evaluation/ClusteringEvaluator.scala
---
@@ -64,12 +65,12 @@ class ClusteringEvaluator @Since("2.3.0")
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21550
**[Test build #91771 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91771/testReport)**
for PR 21550 at commit
[`5d53ec7`](https://github.com/apache/spark/commit/5
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21549#discussion_r195153892
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/xml/UDFXPathUtilSuite.scala
---
@@ -77,6 +77,23 @@ class UDFXPathUtilSui
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/20629
cc @sethah
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spar
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21550
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/21550
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91771/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21549
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/3990/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21549
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/21549
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/21549
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/99/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21549
**[Test build #91784 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91784/testReport)**
for PR 21549 at commit
[`52e706b`](https://github.com/apache/spark/commit/52
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/14653
@evanyc15 would you mind closing this PR? Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For add
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21109
**[Test build #91785 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91785/testReport)**
for PR 21109 at commit
[`fb99390`](https://github.com/apache/spark/commit/fb
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21321#discussion_r195156323
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/FeatureHasher.scala ---
@@ -208,7 +208,8 @@ class FeatureHasher(@Since("2.3.0") override val uid:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21549
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/21549
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91784/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21549
**[Test build #91784 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91784/testReport)**
for PR 21549 at commit
[`52e706b`](https://github.com/apache/spark/commit/5
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21499
Thank you for fixing this! In Spark 3.0, we might introduce a mode for
strict ANSI compliance. We can handle the cases in
https://github.com/apache/spark/pull/20350
---
---
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/21428
LGTM pending tests
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21450#discussion_r195165247
--- Diff: launcher/src/main/java/org/apache/spark/launcher/Main.java ---
@@ -101,6 +99,23 @@ public static void main(String[] argsArray) throws
Exception {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21441
**[Test build #91776 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91776/testReport)**
for PR 21441 at commit
[`da92e01`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21441
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
201 - 300 of 653 matches
Mail list logo