-
Notifications
You must be signed in to change notification settings - Fork 916
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
Host compression #17656
Open
vuule
wants to merge
46
commits into
rapidsai:branch-25.02
Choose a base branch
from
vuule:high-lvl-comp-api
base: branch-25.02
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Host compression #17656
Changes from 39 commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
81dcfa6
random clean up
vuule 4f7794d
jesus
vuule 3166acb
Merge branch 'branch-25.02' into comp-headers-cleanup
vuule b3f03e8
style
vuule 53205c5
style
vuule 05d07ba
Merge branch 'branch-25.02' into comp-headers-cleanup
vuule 7d23502
Merge branch 'branch-25.02' into comp-headers-cleanup
vuule 324d635
Update cpp/src/io/comp/common.hpp
vuule 350db40
Merge branch 'branch-25.02' into comp-headers-cleanup
vuule 0cf8375
Merge branch 'branch-25.02' into comp-headers-cleanup
vuule 947fbd4
fix
vuule 0a64f1c
Merge branch 'comp-headers-cleanup' of https://github.com/vuule/cudf …
vuule 54d9bb9
fix some more
vuule 2ca535b
Merge branch 'branch-25.02' of https://github.com/rapidsai/cudf into …
vuule 963f066
avoid part of nvcomp enabled checks in writers
vuule e119ad8
single-threaded host comp
vuule 3ab8c41
now with more threads
vuule a14b351
decouple orc writer from nvcomp
vuule a8e1dec
Merge branch 'branch-25.02' of https://github.com/rapidsai/cudf into …
vuule 0b6b72d
decouple pq writer from nvcomp
vuule d83abac
missed DEFLATE
vuule e2dce81
simplify
vuule 9b8fc71
clean up
vuule 7aaf5ed
fix
vuule 9a5ca7d
Merge branch 'branch-25.02' into high-lvl-comp-api
vuule 3d311ae
style
vuule ba5321a
Merge branch 'high-lvl-comp-api' of https://github.com/vuule/cudf int…
vuule e010f9f
style some more
vuule 19cd311
Merge branch 'branch-25.02' into high-lvl-comp-api
vuule 3094173
Merge branch 'branch-25.02' into high-lvl-comp-api
vuule b83c1ff
Merge branch 'branch-25.02' of https://github.com/rapidsai/cudf into …
vuule 8ceecff
handle AUTO compression in options
vuule 7fa6055
Merge branch 'high-lvl-comp-api' of https://github.com/vuule/cudf int…
vuule b2cdcf4
Merge branch 'branch-25.02' into high-lvl-comp-api
vuule b5b06aa
style
vuule bfae53a
Merge branch 'high-lvl-comp-api' of https://github.com/vuule/cudf int…
vuule 11ca033
Merge branch 'branch-25.02' into high-lvl-comp-api
vuule ea04f43
Merge branch 'branch-25.02' into high-lvl-comp-api
vuule 3e403b0
Merge branch 'branch-25.02' into high-lvl-comp-api
vuule ae1b980
remove unused function
vuule a725970
Merge branch 'branch-25.02' into high-lvl-comp-api
vuule 05b5f3d
code review suggestions
vuule 70baa8d
Merge branch 'high-lvl-comp-api' of https://github.com/vuule/cudf int…
vuule b81c7f5
Merge branch 'branch-25.02' into high-lvl-comp-api
vuule c72b66d
env var; limit num streams
vuule 559ca43
Merge branch 'high-lvl-comp-api' of https://github.com/vuule/cudf int…
vuule File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For my education, Is it common sense that
AUTO
should be justSNAPPY
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
compression_type is common for all file formats, so AUTO may mean different compression types for different formats.
I guess we could remove AUTO and set a concrete compression type as the default for each format.