[FEA] Add DELTA_BINARY_PACKED decoding support to Parquet reader #13637
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…eature/delta_binary
|
Pull requests from external contributors require approval from a |
github-actions
bot
added
libcudf
vuule
added
feature request
|
/ok to test |
nvdbaranec
requested changes
Aug 17, 2023
Co-authored-by: nvdbaranec <[email protected]>
nvdbaranec
requested changes
Aug 22, 2023
zhuoxunyi
referenced
this pull request
Aug 23, 2023
Fixes: #13864 This PR fixes an issue with `loc` indexer where some special handling needs to be done when `columns` is of type `MultiIndex`. Authors: - GALI PREM SAGAR (https://github.com/galipremsagar) Approvers: - Lawrence Mitchell (https://github.com/wence-) URL: #13929
nvdbaranec
approved these changes
Aug 23, 2023
galipremsagar
approved these changes
Aug 23, 2023
Co-authored-by: GALI PREM SAGAR <[email protected]>
|
/ok to test |
|
/merge |
rapids-bot bot
pushed a commit
that referenced
this pull request
Sep 13, 2023
#13637 added a static stream pool object for use by the Parquet reader. This PR expands upon that by: - Moving the stream pool to the `cudf::detail` namespace. - Adding a debugging implementation that always returns the default stream. - Hiding implementation details behind a more streamlined interface. - Using cuda events for synchronization. Authors: - Ed Seidl (https://github.com/etseidl) - Vukasin Milovanovic (https://github.com/vuule) - Mark Harris (https://github.com/harrism) Approvers: - Bradley Dice (https://github.com/bdice) - Vukasin Milovanovic (https://github.com/vuule) - Mark Harris (https://github.com/harrism) URL: #13922
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Part of #13501. This adds support for decoding Parquet pages that are DELTA_BINARY_PACKED.
In addition to adding delta support, this PR incorporates changes introduced in #13622, such as using a mask to determine which decoding kernels to run, and adding parameters to the
page_state_buffers_sstruct to reduce the amount of shared memory used.Checklist