Skip to content
New issue

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

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

Already on GitHub? # to your account

Support casting Map to String #4785

Merged

Conversation

HaoYang670
Copy link
Collaborator

@HaoYang670 HaoYang670 commented Feb 15, 2022

Signed-off-by: remzi 13716567376yh@gmail.com
close #4696

In this PR, we:

  1. refactor the function cast_array_to_string by abstracting the logic of concatenating String Array to a new function concatenateStringArrayElements which is also used by cast_map_to_string
  2. create the function cast_map_to_string which casts MapType column to StringType column. Also add some tests.
  3. deleted some outdated tests.

Signed-off-by: remzi <13716567376yh@gmail.com>
Signed-off-by: remzi <13716567376yh@gmail.com>
Signed-off-by: remzi <13716567376yh@gmail.com>
Signed-off-by: remzi <13716567376yh@gmail.com>
Signed-off-by: remzi <13716567376yh@gmail.com>
Signed-off-by: remzi <13716567376yh@gmail.com>
Signed-off-by: remzi <13716567376yh@gmail.com>
Signed-off-by: remzi <13716567376yh@gmail.com>
@HaoYang670
Copy link
Collaborator Author

build

@HaoYang670 HaoYang670 self-assigned this Feb 15, 2022
Signed-off-by: remzi <13716567376yh@gmail.com>
Signed-off-by: remzi <13716567376yh@gmail.com>
@HaoYang670
Copy link
Collaborator Author

build

@HaoYang670 HaoYang670 merged commit d90c6d7 into NVIDIA:branch-22.04 Feb 17, 2022
@HaoYang670 HaoYang670 deleted the issue4696_cast_map_to_string branch February 17, 2022 01:07
@sameerz sameerz added the feature request New feature or request label Feb 18, 2022
@sameerz sameerz added this to the Feb 14 - Feb 25 milestone Feb 18, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
feature request New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEA] Support casting map to string
3 participants