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

[BUG] String corruption when skipping rows #14560

Closed
etseidl opened this issue Dec 4, 2023 · 0 comments
Closed

[BUG] String corruption when skipping rows #14560

etseidl opened this issue Dec 4, 2023 · 0 comments
Labels
bug Something isn't working

Comments

@etseidl
Copy link
Contributor

etseidl commented Dec 4, 2023

Spark-rapids uncovered an error in string processing NVIDIA/spark-rapids#9948. The issue arises when lists are present, num_rows/skip_rows is utilized, and the string column contains rows that begin and end in different pages. In this particular case, the code used to calculate page start/end value indices incorrectly set the start value to 0, resulting in an overestimation of the string buffer size, and the introduction of uninitialized data to the resulting column.

This condition is hard to test for because libcudf will always start a page at a new row boundary.

@etseidl etseidl added Needs Triage Need team to review and classify bug Something isn't working labels Dec 4, 2023
@bdice bdice removed the Needs Triage Need team to review and classify label Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests

2 participants