Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEA] Improve CreateMap to support multiple key and value expressions #3229

Closed
andygrove opened this issue Aug 13, 2021 · 0 comments · Fixed by #3251
Closed

[FEA] Improve CreateMap to support multiple key and value expressions #3229

andygrove opened this issue Aug 13, 2021 · 0 comments · Fixed by #3251
Assignees
Labels
cudf_dependency An issue or PR with this label depends on a new feature in cudf feature request New feature or request

Comments

@andygrove
Copy link
Contributor

andygrove commented Aug 13, 2021

Is your feature request related to a problem? Please describe.
Issue #3014 adds basic support for CreateMap from two expressions. This issue is to extend support for multiple expressions e.g. select map(key1, value1, key2, value2) from foo.

Describe the solution you'd like
We either need to implement this in cuDF or in the plugin. We could concatenate the columns into a single pair of columns before calling cuDF.

Describe alternatives you've considered
N/A

Additional context
This is part of #44

@andygrove andygrove added feature request New feature or request ? - Needs Triage Need team to review and classify labels Aug 13, 2021
@andygrove andygrove added this to the Aug 16 - Aug 27 milestone Aug 13, 2021
@andygrove andygrove self-assigned this Aug 13, 2021
@Salonijain27 Salonijain27 added cudf_dependency An issue or PR with this label depends on a new feature in cudf and removed ? - Needs Triage Need team to review and classify labels Aug 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cudf_dependency An issue or PR with this label depends on a new feature in cudf feature request New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants