Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22527
Looks good to me too!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22527
Hey @ueshin, sorry I was late. I'm a bit busy for these couple of weeks so
please don't block by me and ignore me. Thank you for asking it to me.
---
--
Github user michalsenkyr commented on the issue:
https://github.com/apache/spark/pull/22527
Thanks! I created a new PR with array, list and map support.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22527
Seems like there is a merge commit in apache git
https://git-wip-us.apache.org/repos/asf?p=spark.git, but not in GitHub yet.
---
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22527
Sorry, the merge script failed. Let me try again a while later.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22527
I'd merge this not to block the following prs to support array/list and map
of beans.
---
-
To unsubscribe, e-mail: reviews-unsub
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22527
Thanks! merging to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22527
**[Test build #96954 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96954/testReport)**
for PR 22527 at commit
[`e9b5a98`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22527
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96954/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22527
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/22527
**[Test build #96954 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96954/testReport)**
for PR 22527 at commit
[`e9b5a98`](https://github.com/apache/spark/commit/e9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22527
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96901/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22527
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/22527
**[Test build #96901 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96901/testReport)**
for PR 22527 at commit
[`3b2a431`](https://github.com/apache/spark/commit/3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22527
**[Test build #96901 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96901/testReport)**
for PR 22527 at commit
[`3b2a431`](https://github.com/apache/spark/commit/3b
Github user michalsenkyr commented on the issue:
https://github.com/apache/spark/pull/22527
@ueshin Yes. I am already working on array/list support. Will add maps as
well. It shouldn't require a rewrite now that the code is restructured, just
new cases in pattern match. So I think it'
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22527
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/22527
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96869/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22527
**[Test build #96869 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96869/testReport)**
for PR 22527 at commit
[`3fe63c8`](https://github.com/apache/spark/commit/3
Github user michalsenkyr commented on the issue:
https://github.com/apache/spark/pull/22527
I restructured the code in this commit to allow easier addition of
array/list support in the future.
---
-
To unsubscribe,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22527
**[Test build #96869 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96869/testReport)**
for PR 22527 at commit
[`3fe63c8`](https://github.com/apache/spark/commit/3f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22527
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/22527
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96809/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22527
**[Test build #96809 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96809/testReport)**
for PR 22527 at commit
[`d8083cf`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22527
**[Test build #96809 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96809/testReport)**
for PR 22527 at commit
[`d8083cf`](https://github.com/apache/spark/commit/d8
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/22527
Jenkins, ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22527
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22527
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22527
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
29 matches
Mail list logo