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] simplify page_state_s and possibly other structures for specialized parquet kernels #15267

Open
abellina opened this issue Mar 11, 2024 · 0 comments
Labels
cuIO cuIO issue feature request New feature or request libcudf Affects libcudf (C++/CUDA) code. Performance Performance related issue

Comments

@abellina
Copy link
Contributor

@vuule brought up the point here #15159 (comment) that page_state_s could be simplified for the fixed/fixed dictionary kernels, as they don't need all the members of this structure and it might help with shared memory use.

I also think there are other shared structures that we should audit and clean up. I am filing this issue so we don't loose track of it.

@abellina abellina added the feature request New feature or request label Mar 11, 2024
@GregoryKimball GregoryKimball added libcudf Affects libcudf (C++/CUDA) code. cuIO cuIO issue Performance Performance related issue labels Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cuIO cuIO issue feature request New feature or request libcudf Affects libcudf (C++/CUDA) code. Performance Performance related issue
Projects
Status: In Progress
Status: No status
Development

No branches or pull requests

2 participants