Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22544
@cloud-fan Sounds good Wenchen. Thanks for the detailed analysis.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apa
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22544
After thinking more, I'm going it to merge it to 2.4 directly, because:
1. `ImplicitTypeCasts` handles array type but not map type, which looks a
bug to me
2. I checked all the implementati
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22544
@cloud-fan @ueshin @viirya Thank you very much.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additi
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22544
I'm merging it to master, please send another PR to 2.4, without touching
the `findTightestCommonType`. Thanks!
---
-
To unsub
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22544
@cloud-fan Are we targeting this for 2.4 ? If so, i was wondering if we
should make a restricted fix that affects only this function by adding a case
in `FunctionArgumentConversion` for 2.4 whil
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22544
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96660/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22544
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/22544
**[Test build #96660 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96660/testReport)**
for PR 22544 at commit
[`69c78d7`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22544
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/22544
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96644/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22544
**[Test build #96644 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96644/testReport)**
for PR 22544 at commit
[`9b12969`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22544
**[Test build #96660 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96660/testReport)**
for PR 22544 at commit
[`69c78d7`](https://github.com/apache/spark/commit/69
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22544
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/22544
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/3511/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22544
**[Test build #96644 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96644/testReport)**
for PR 22544 at commit
[`9b12969`](https://github.com/apache/spark/commit/9b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22544
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/3498/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22544
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/22544
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96620/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22544
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/22544
**[Test build #96620 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96620/testReport)**
for PR 22544 at commit
[`2f224f2`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22544
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96611/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22544
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/22544
**[Test build #96611 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96611/testReport)**
for PR 22544 at commit
[`48eb0ff`](https://github.com/apache/spark/commit/4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22544
**[Test build #96620 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96620/testReport)**
for PR 22544 at commit
[`2f224f2`](https://github.com/apache/spark/commit/2f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22544
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/22544
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/3481/
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22544
cc @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@sp
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22544
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/3474/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22544
**[Test build #96611 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96611/testReport)**
for PR 22544 at commit
[`48eb0ff`](https://github.com/apache/spark/commit/48
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22544
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/22544
retest this please.
---
-
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/22544
**[Test build #96603 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96603/testReport)**
for PR 22544 at commit
[`48eb0ff`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22544
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/22544
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96603/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22544
**[Test build #96603 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96603/testReport)**
for PR 22544 at commit
[`48eb0ff`](https://github.com/apache/spark/commit/48
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22544
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/22544
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/3467/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22544
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/22544
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96544/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22544
**[Test build #96544 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96544/testReport)**
for PR 22544 at commit
[`4d32f2c`](https://github.com/apache/spark/commit/4
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22544
@dilipbiswal The two examples shown in the PR description are allowed in
current and this proposes to disallow them? I'm a bit confused by the
description.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22544
**[Test build #96544 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96544/testReport)**
for PR 22544 at commit
[`4d32f2c`](https://github.com/apache/spark/commit/4d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22544
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/3430/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22544
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
44 matches
Mail list logo