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

Replace GPU-unsupported \z with an alternative RLIKE expression #10090

Merged
merged 5 commits into from
Dec 26, 2023

Conversation

gerashegalov
Copy link
Collaborator

@gerashegalov gerashegalov commented Dec 21, 2023

FIxes #9992

replaced a broken and unsupported use of \z regex with another very contrived condition testing the laziness of the conditional with side effects evaluation.

Signed-off-by: Gera Shegalov gera@apache.org

Signed-off-by: Gera Shegalov <gera@apache.org>
Signed-off-by: Gera Shegalov <gera@apache.org>
@gerashegalov gerashegalov changed the title Replace GPU-unsupported \z with an alternative RLIKE expressions Replace GPU-unsupported \z with an alternative RLIKE expression Dec 21, 2023
@gerashegalov
Copy link
Collaborator Author

build

@gerashegalov gerashegalov marked this pull request as ready for review December 21, 2023 22:45
@gerashegalov gerashegalov self-assigned this Dec 22, 2023
@gerashegalov gerashegalov added test Only impacts tests bug Something isn't working labels Dec 22, 2023
@revans2 revans2 merged commit e09c144 into NVIDIA:branch-24.02 Dec 26, 2023
38 checks passed
@gerashegalov gerashegalov deleted the gerashegalov/issue9992 branch December 26, 2023 19:01
gerashegalov added a commit to gerashegalov/spark-rapids that referenced this pull request Dec 28, 2023
Fixes NVIDIA#9992

Addendum to NVIDIA#10090

Signed-off-by: Gera Shegalov <gera@apache.org>
gerashegalov added a commit that referenced this pull request Dec 28, 2023
…ast (#10112)

Fixes #9992

Addendum to #10090

Signed-off-by: Gera Shegalov <gera@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working test Only impacts tests
Projects
None yet
2 participants