-
Notifications
You must be signed in to change notification settings - Fork 842
Add Zstandard compression support and update tests #12201
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
Open
JakeChampion
wants to merge
31
commits into
apache:master
Choose a base branch
from
JakeChampion:zstd
base: master
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
Changes from 21 commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
b22cbee
Add comprehensive zstd compression support to ATS
JakeChampion facf384
Fix Accept-Encoding cache matching and update test expectations
JakeChampion eda7c29
Fix case sensitivity in ZSTD package find command
JakeChampion 21e27bf
Add ZSTD version output to compile-time features
JakeChampion 2a2acf6
Add ZSTD support to Accept-Encoding and Content-Encoding headers
JakeChampion 536f920
Add ZSTD length definition to HTTP value initialization
JakeChampion 9a5432d
Update ZSTD compression level from 6 to 10 for improved performance
JakeChampion ab79771
compress: Add configurable compression levels
JakeChampion d94321d
Update compress.gold
JakeChampion 729fbac
Update compress.gold
JakeChampion 4bf0c10
Merge branch 'master' into zstd
JakeChampion 17e3c6b
Use upstream zstd CMake; remove Findzstd.cmake
JakeChampion 91029c5
Ensure HAVE_ZSTD_H is always set to a value
JakeChampion c7389c5
place HAVE_ZSTD_H nearer the other cmakedefine calls
JakeChampion 4608bad
Apply suggestions from code review
JakeChampion 03e647a
Renames ZSTD compression functions for consistency
JakeChampion 079b002
Merge remote-tracking branch 'upstream/master' into zstd
JakeChampion c953863
Merge remote-tracking branch 'upstream/master' into zstd
JakeChampion 0bb4fb5
Add flush handling to zstd compression stream
JakeChampion d31c21b
refactor zstd compression to reduce code duplication
JakeChampion 3c37f08
Update compress.gold
JakeChampion 7668be8
Updates Accept-Encoding variability check logic
JakeChampion 88a02b2
Remove duplicate luajit package from Dockerfile dependencies
JakeChampion 8e4f9c8
only include zstd in error message if zstd exists
JakeChampion 23716b1
Merge remote-tracking branch 'upstream/master' into zstd
JakeChampion e03fa5c
update zstd compression context initialization to return a success st…
JakeChampion b87c864
input validation for zstd compression
JakeChampion 1b1ed1c
input validation for zstd compression
JakeChampion 6fa64e3
replace TSError with error for Zstandard compression context failure
JakeChampion c1ef84e
Merge remote-tracking branch 'upstream/master' into zstd
JakeChampion 6d440f1
revert content encoding quality factor calculation
JakeChampion 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
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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.
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.
Uh oh!
There was an error while loading. Please reload this page.