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.

Sign up for GitHub

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

Jump to bottom

Use vectorized unpacking in ColumnLevelDecoderImpl for ColumnLevelDecoderImpl #2153

Open

Ted-Jiang opened this issue Jul 24, 2022 · 1 comment

Member

Ted-Jiang commented Jul 24, 2022

It might be faster to decode to a temporary buffer, to allow vectorized unpacking, but definitely something that can be done as a follow up

Originally posted by @tustvold in #2111 (comment)

Ted-Jiang mentioned this issue

Support skip_def_levels for ColumnLevelDecoder #2111

Merged

Contributor

tustvold commented Jul 24, 2022

FWIW it would be even more optimal to push the skip down to the RLEDecoder, but that might be more complex to implement

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment