Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71952/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71952 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71952/testReport)**
for PR 16605 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71952 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71952/testReport)**
for PR 16605 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71940/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71940 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71940/testReport)**
for PR 16605 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71939/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71939 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71939/testReport)**
for PR 16605 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71940 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71940/testReport)**
for PR 16605 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71939 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71939/testReport)**
for PR 16605 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71935/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71935 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71935/testReport)**
for PR 16605 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71934 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71934/testReport)**
for PR 16605 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71935 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71935/testReport)**
for PR 16605 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71934/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71934 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71934/testReport)**
for PR 16605 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71729/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71729 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71729/testReport)**
for PR 16605 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71729 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71729/testReport)**
for PR 16605 at commit
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/16605
okay, I'll update this pr in that way, thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16605
SGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/16605
The workaround @cloud-fan said is good to me though, IMO the most critical
issue here is that this cast exception happens not in analysis phases but in
runtime. So, at least I think we should modify
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/16605
okay. But, if this issue finished, I'm planning to take SPARK-12823 in a
similar way.
Do u think also it's not also worth trying struct? cc: @cloud-fan
---
If your project is set up for it,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71631/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71631 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71631/testReport)**
for PR 16605 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16605
Well, it will be good if we can support `Array` in `ScalaUDF`, but it's not
a big deal as users can easily do `udf { (seq: Seq[Int]) => val a =
seq.toArray; // do anything you like with the array
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16605
Sure, @maropu . `WrappedArray` is not documented for now.
Hi, @gatorsmile and @cloud-fan .
Could you review this PR?
---
If your project is set up for it, you can reply to this
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/16605
oh, yea. I didn't find that and I think it's a good point.
IMO `WrappedArray` is implicitly used inside for implicit conversions, so
users do not use `WrappedArray` directly for UDFs in most
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71631 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71631/testReport)**
for PR 16605 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16605
Hi, @maropu .
In the current master with your example, we can do the following. How do
you think about this?
```scala
scala> import scala.collection.mutable.WrappedArray
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71570/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71570 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71570/testReport)**
for PR 16605 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71568/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71568 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71568/testReport)**
for PR 16605 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71570 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71570/testReport)**
for PR 16605 at commit
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/16605
@dongjoon-hyun okay, I applied your comments into this pr. Could you check
again to satisfy your intentions?
---
If your project is set up for it, you can reply to this email and have your
reply
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71568 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71568/testReport)**
for PR 16605 at commit
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/16605
many thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16605
Sure, @maropu . I'll do that tomorrow morning (PST).
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/16605
@dongjoon-hyun Could you take time to review this before committers do?
Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71468/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71468 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71468/testReport)**
for PR 16605 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71468 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71468/testReport)**
for PR 16605 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71456/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71456 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71456/testReport)**
for PR 16605 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71454/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71454 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71454/testReport)**
for PR 16605 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71456 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71456/testReport)**
for PR 16605 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71454 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71454/testReport)**
for PR 16605 at commit
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/16605
I'm looking for another approach not to break backward compatibility...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71449/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16605
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71449 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71449/testReport)**
for PR 16605 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16605
**[Test build #71449 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71449/testReport)**
for PR 16605 at commit
65 matches
Mail list logo