Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22696
LGTM
Thanks! Merged to master/2.4
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22696
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/22696
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97289/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22696
**[Test build #97289 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97289/testReport)**
for PR 22696 at commit
[`b0dc140`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22696
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/3908/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22696
**[Test build #97289 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97289/testReport)**
for PR 22696 at commit
[`b0dc140`](https://github.com/apache/spark/commit/b0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22696
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/22696
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/22696
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97280/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22696
**[Test build #97280 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97280/testReport)**
for PR 22696 at commit
[`78a1689`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22696
**[Test build #97280 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97280/testReport)**
for PR 22696 at commit
[`78a1689`](https://github.com/apache/spark/commit/78
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22696
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/3900/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22696
Merged build finished. Test PASSed.
---
-
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/22696
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/22696
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97276/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22696
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/22696
**[Test build #97276 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97276/testReport)**
for PR 22696 at commit
[`78a1689`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22696
**[Test build #97276 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97276/testReport)**
for PR 22696 at commit
[`78a1689`](https://github.com/apache/spark/commit/78
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22696
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/3896/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22696
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22696
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/22696
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/22696
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97266/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22696
**[Test build #97266 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97266/testReport)**
for PR 22696 at commit
[`78a1689`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22696
**[Test build #97266 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97266/testReport)**
for PR 22696 at commit
[`78a1689`](https://github.com/apache/spark/commit/78
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22696
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/22696
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/3889/
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22696
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22696
I think we should mention this in migration guide. Although previous
behavior is wrong, it might be treated as a "feature" of Spark SQL. We should
explicitly let users know this change.
---
--
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22696
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/22696
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97254/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22696
**[Test build #97254 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97254/testReport)**
for PR 22696 at commit
[`f6bbd38`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22696
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97252/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22696
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/22696
**[Test build #97252 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97252/testReport)**
for PR 22696 at commit
[`f33400d`](https://github.com/apache/spark/commit/f
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/22696
I added the release-notes label to the JIRA ticket. I am not sure if there
is a migration-guide label.
---
-
To unsubscribe,
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22696
nice catch! Shall we mention this in the migration guide? It is a behavior
change (despite the previous was a wrong behavior), so I think warning users
might be a good thing. LGTM otherwise.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22696
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/3883/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22696
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/22696
**[Test build #97254 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97254/testReport)**
for PR 22696 at commit
[`f6bbd38`](https://github.com/apache/spark/commit/f6
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22696
cc @hvanhovell @gatorsmile @viirya @mgaido91 @ueshin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For ad
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22696
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/3881/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22696
**[Test build #97252 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97252/testReport)**
for PR 22696 at commit
[`f33400d`](https://github.com/apache/spark/commit/f3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22696
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
44 matches
Mail list logo