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

CreateMap support for multiple key-value pairs #3251

Merged
merged 12 commits into from
Aug 25, 2021

Conversation

andygrove
Copy link
Contributor

Builds on #3230.

Closes #3229.

There is a follow-on issue #3250 for making duplicate handling compatible with Spark so that we can remove the config option and enable CreateMap by default.

Signed-off-by: Andy Grove <andygrove@nvidia.com>
Signed-off-by: Andy Grove <andygrove@nvidia.com>
@andygrove andygrove force-pushed the gpu-create-map-multi branch from 76872b7 to cc01937 Compare August 19, 2021 15:23
Signed-off-by: Andy Grove <andygrove@nvidia.com>
Signed-off-by: Andy Grove <andygrove@nvidia.com>
@sameerz sameerz added the feature request New feature or request label Aug 19, 2021
@sameerz sameerz added this to the Aug 16 - Aug 27 milestone Aug 19, 2021
@andygrove andygrove changed the title WIP: CreateMap support for multiple key-value pairs CreateMap support for multiple key-value pairs Aug 19, 2021
@andygrove andygrove marked this pull request as ready for review August 19, 2021 18:31
@andygrove andygrove requested a review from revans2 August 24, 2021 14:05
revans2
revans2 previously approved these changes Aug 24, 2021
Copy link
Collaborator

@revans2 revans2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a nit

@andygrove
Copy link
Contributor Author

build

@andygrove andygrove merged commit 66c088e into NVIDIA:branch-21.10 Aug 25, 2021
@andygrove andygrove deleted the gpu-create-map-multi branch August 25, 2021 00:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEA] Improve CreateMap to support multiple key and value expressions
4 participants