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

[Backport 2.x] Add hardware-accelerated codecs for DEFLATE and LZ4 (#122) #150

Merged
merged 1 commit into from
May 30, 2024

Conversation

mulugetam
Copy link
Contributor

  • Add QAT accelerated compression.

Signed-off-by: Mulugeta Mammo mulugeta.mammo@intel.com

  • Use own classes for QAT codec. Apply SpotlessJavaCheck.

Signed-off-by: Mulugeta Mammo mulugeta.mammo@intel.com

  • Declare fields final, unless required not to. Throw a valid type of exception.

Signed-off-by: Mulugeta Mammo mulugeta.mammo@intel.com

  • Use assumeThat in the Qat test classes.

Signed-off-by: Mulugeta Mammo mulugeta.mammo@intel.com

  • Add more QAT availability check in QatCodecTests.

Signed-off-by: Mulugeta Mammo mulugeta.mammo@intel.com

  • Make LZ4 the default algorithm for QAT.

Signed-off-by: Mulugeta Mammo mulugeta.mammo@intel.com

  • Make 'auto' the default execution mode for QAT. Also, minor clean up work.

Signed-off-by: Mulugeta Mammo mulugeta.mammo@intel.com

  • Revert compression level for ZSTD to 3.

Signed-off-by: Mulugeta Mammo mulugeta.mammo@intel.com

  • Replace QatLz4/DeflateCompressionMode classes with QatCompressionMode.

Signed-off-by: Mulugeta Mammo mulugeta.mammo@intel.com

  • Fix a MultiCodecMergeIT test fail.

Signed-off-by: Mulugeta Mammo mulugeta.mammo@intel.com

  • Remove hard-coded values for default compression level.

Signed-off-by: Mulugeta Mammo mulugeta.mammo@intel.com


Signed-off-by: Mulugeta Mammo mulugeta.mammo@intel.com
Signed-off-by: mulugetam mulugeta.mammo@intel.com
Co-authored-by: Mulugeta Mammo cppx86@gmail.com
(cherry picked from commit c8b0d80)

Description

Describe what this change achieves.

Issues Resolved

List any issues this PR will resolve, e.g. Closes [...].

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

…ct#122)

* Add QAT accelerated compression.

Signed-off-by: Mulugeta Mammo <mulugeta.mammo@intel.com>

* Use own classes for QAT codec. Apply SpotlessJavaCheck.

Signed-off-by: Mulugeta Mammo <mulugeta.mammo@intel.com>

* Declare fields final, unless required not to. Throw a valid type of exception.

Signed-off-by: Mulugeta Mammo <mulugeta.mammo@intel.com>

* Use assumeThat in the Qat test classes.

Signed-off-by: Mulugeta Mammo <mulugeta.mammo@intel.com>

* Add more QAT availability check in QatCodecTests.

Signed-off-by: Mulugeta Mammo <mulugeta.mammo@intel.com>

* Make LZ4 the default algorithm for QAT.

Signed-off-by: Mulugeta Mammo <mulugeta.mammo@intel.com>

* Make 'auto' the default execution mode for QAT. Also, minor clean up work.

Signed-off-by: Mulugeta Mammo <mulugeta.mammo@intel.com>

* Revert compression level for ZSTD to 3.

Signed-off-by: Mulugeta Mammo <mulugeta.mammo@intel.com>

* Replace QatLz4/DeflateCompressionMode classes with QatCompressionMode.

Signed-off-by: Mulugeta Mammo <mulugeta.mammo@intel.com>

* Fix a MultiCodecMergeIT test fail.

Signed-off-by: Mulugeta Mammo <mulugeta.mammo@intel.com>

* Remove hard-coded values for default compression level.

Signed-off-by: Mulugeta Mammo <mulugeta.mammo@intel.com>

---------

Signed-off-by: Mulugeta Mammo <mulugeta.mammo@intel.com>
Signed-off-by: mulugetam <mulugeta.mammo@intel.com>
Co-authored-by: Mulugeta Mammo <cppx86@gmail.com>
(cherry picked from commit c8b0d80)
@mulugetam mulugetam changed the title Add hardware-accelerated codecs for DEFLATE and LZ4 (#122) [Backport 2.x] Add hardware-accelerated codecs for DEFLATE and LZ4 (#122) May 30, 2024
@reta reta merged commit 75c1a2e into opensearch-project:2.x May 30, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants