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

[BUG] RegExp transpiler fails to detect some choice expressions that cuDF cannot compile #4524

Closed
andygrove opened this issue Jan 13, 2022 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@andygrove
Copy link
Contributor

andygrove commented Jan 13, 2022

Describe the bug
During fuzz testing I found some patterns that the transpiler allows but cuDF does not support, such as [^\r\n]3$|}}\t. This is not valid because the left side of the choice ends with a line anchor, causing cuDF to fail during the compilation of the pattern.

Steps/Code to reproduce bug
Add this pattern to the "transpiler detects invalid cuDF patterns" test in RegularExpressionTranspilerSuite.

Expected behavior
Test should pass.

Environment details (please complete the following information)
N/A

Additional context
None

@andygrove andygrove added bug Something isn't working ? - Needs Triage Need team to review and classify labels Jan 13, 2022
@andygrove andygrove added this to the Jan 10 - Jan 28 milestone Jan 13, 2022
@andygrove andygrove self-assigned this Jan 13, 2022
@andygrove
Copy link
Contributor Author

False alarm. This was only happening in a specific branch.

@sameerz sameerz removed the ? - Needs Triage Need team to review and classify label Jan 18, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants