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/SparkPullRequestBuilder/91773/
Test PASSed.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21529
**[Test build #91773 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91773/testReport)**
for PR 21529 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21503#discussion_r195170789
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Relation.scala
---
@@ -32,79 +31,35 @@ import
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/91777/
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21441
**[Test build #91777 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91777/testReport)**
for PR 21441 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21549
**[Test build #91786 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91786/testReport)**
for PR 21549 at commit
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
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/100/
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/3991/
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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21549
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/21441
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91776/
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
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
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 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:
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 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
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
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
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
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
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
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
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 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
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/
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/21550
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
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:
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
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
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 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 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
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
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
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 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
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:
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 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 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 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
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 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
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:
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
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/
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
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
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/
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
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
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
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
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
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
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
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
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 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
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
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 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 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:
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
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
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/
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
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
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 {
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:
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 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/
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
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
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
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
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:
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
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 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
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 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
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/
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
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/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/
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/21260
Merged build finished. Test PASSed.
---
-
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/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 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
301 - 400 of 654 matches
Mail list logo