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] Support regular expression delimiters for str_to_map #4721

Closed
ttnghia opened this issue Feb 8, 2022 · 0 comments · Fixed by #4636
Closed

[FEA] Support regular expression delimiters for str_to_map #4721

ttnghia opened this issue Feb 8, 2022 · 0 comments · Fixed by #4636
Labels
cudf_dependency An issue or PR with this label depends on a new feature in cudf feature request New feature or request P1 Nice to have for release SQL part of the SQL/Dataframe plugin

Comments

@ttnghia
Copy link
Collaborator

ttnghia commented Feb 8, 2022

The str_to_map SQL function performs string splitting on the input to generate the output map. For literal delimiters this is trivial. For regular expression delimiter, this requires a lot of effort. We should be able to support regular expression delimiters as much as possible.

Relevant PR:

Related PR and issues:

@ttnghia ttnghia added feature request New feature or request ? - Needs Triage Need team to review and classify labels Feb 8, 2022
@ttnghia ttnghia changed the title [FEA] Support regular expression in str_to_map [FEA] Support regular expression delimiters for str_to_map Feb 8, 2022
@ttnghia ttnghia linked a pull request Feb 8, 2022 that will close this issue
@ttnghia ttnghia added cudf_dependency An issue or PR with this label depends on a new feature in cudf P1 Nice to have for release SQL part of the SQL/Dataframe plugin and removed ? - Needs Triage Need team to review and classify labels Feb 8, 2022
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 P1 Nice to have for release SQL part of the SQL/Dataframe plugin
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant