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

feat(core-clp): Add LZMA Compressor implementation and LZMA dependency. #614

Merged
merged 68 commits into from
Dec 23, 2024
Merged
Show file tree
Hide file tree
Changes from 67 commits
Commits
Show all changes
68 commits
Select commit Hold shift + click to select a range
dceb564
Add lzma download and port lzma scripts
Bill-hbrhbr Nov 25, 2024
d5af274
Make unit test pass
Bill-hbrhbr Nov 25, 2024
b94ca26
Refactor lzma compressor to group common functionalities into helplers
Bill-hbrhbr Nov 26, 2024
707c412
Improve comments
Bill-hbrhbr Nov 27, 2024
6d1ab8f
Fix reference link
Bill-hbrhbr Nov 27, 2024
89b5707
Add install for CentOS
Bill-hbrhbr Nov 27, 2024
c646cea
Apply coderabbit suggestions
Bill-hbrhbr Nov 27, 2024
c91e5fb
Remove decompressor related files
Bill-hbrhbr Nov 27, 2024
26b0663
Address review concerns
Bill-hbrhbr Nov 30, 2024
740bc1c
Address review concern
Bill-hbrhbr Dec 2, 2024
e2be883
Simplify else-if
Bill-hbrhbr Dec 2, 2024
905367d
Fix else-if
Bill-hbrhbr Dec 2, 2024
8ae88b2
Add lzma (xz) dep to MacOS
Bill-hbrhbr Dec 2, 2024
0d0c20e
Refactor helper run_lzma()
Bill-hbrhbr Dec 2, 2024
559485d
Update function doc
Bill-hbrhbr Dec 2, 2024
7c69c69
Clarify unit test early termination
Bill-hbrhbr Dec 2, 2024
a6d68b8
Update components/core/tests/test-StreamingCompression.cpp
Bill-hbrhbr Dec 2, 2024
1519c21
Split LZMA_RUN from flush actions
Bill-hbrhbr Dec 3, 2024
655bb46
Refactor unit test
Bill-hbrhbr Dec 3, 2024
4fb6c01
Update components/core/src/clp/streaming_compression/lzma/Compressor.cpp
Bill-hbrhbr Dec 3, 2024
a8799b5
Merge edits
Bill-hbrhbr Dec 3, 2024
2b85f01
Fix import
Bill-hbrhbr Dec 3, 2024
eda7d6c
Apply suggestions from code review
Bill-hbrhbr Dec 4, 2024
4164a9d
Address review concern
Bill-hbrhbr Dec 4, 2024
8ab0653
Add a comment
Bill-hbrhbr Dec 5, 2024
c436f21
Apply suggestions from code review
Bill-hbrhbr Dec 6, 2024
7bd34d2
Update comment to 100-char length
Bill-hbrhbr Dec 6, 2024
efd2b27
Fix according to coding style guidelines
Bill-hbrhbr Dec 11, 2024
c530f92
Apply suggestions from code review
Bill-hbrhbr Dec 12, 2024
e751ee6
Update CMakeLists.txt
Bill-hbrhbr Dec 12, 2024
1c5efcd
Address review concern
Bill-hbrhbr Dec 12, 2024
856c7cb
Update TODO
Bill-hbrhbr Dec 12, 2024
43e22d2
Case fix
Bill-hbrhbr Dec 12, 2024
829a6b2
Remove unnecessary function inline comments
Bill-hbrhbr Dec 12, 2024
81e1807
Improve comment
Bill-hbrhbr Dec 12, 2024
09b73c7
Refactor lzma stream related functions into a nested helper class
Bill-hbrhbr Dec 17, 2024
7cedb25
Adress coderabbit suggestions
Bill-hbrhbr Dec 19, 2024
3dbe388
feat(clp-package): Add support for deleting archives that are exclusi…
haiqi96 Nov 27, 2024
5d90054
feat(clp-s): Add the write path for single-file archives. (#563)
wraymo Nov 27, 2024
2b88c6f
test: Allow multiple trials when unittesting http headers (#613)
anlowee Nov 28, 2024
290ede3
chore(log-viewer-webui): Update `yscope-log-viewer` to the latest ver…
junhaoliao Nov 29, 2024
2c0e053
test(clp-s): Add end-to-end test case for compression and extraction.…
AVMatthews Dec 2, 2024
cbf8bf9
docs(clp-json): Update list of characters that requires escaping in q…
gibber9809 Dec 2, 2024
44b0f2b
feat(core): Add `ErrorCode` template to standardize conversion of use…
LinZhihao-723 Dec 2, 2024
4d21d9b
revert(core): Remove temporary output directory option from `clp` and…
haiqi96 Dec 5, 2024
36892c1
refactor(clp-package): Unify the metadata schema for JSON and IR stre…
haiqi96 Dec 6, 2024
604bd75
feat(clp-s): Add command line options for stubbed out kv-pair-IR inge…
AVMatthews Dec 9, 2024
0a9322b
feat(ffi): Add initial implementation of `IrErrorCode` (using the `Er…
LinZhihao-723 Dec 9, 2024
78a535c
feat(ffi): Add support for auto/user-generated KV-pairs in `KeyValueP…
LinZhihao-723 Dec 9, 2024
42db88c
build(docs): Update dependencies to latest versions. (#631)
kirkrodrigues Dec 10, 2024
13c7528
ci(pr-title-checks): Remove default GH workflow permissions and docum…
kirkrodrigues Dec 12, 2024
8b34dac
feat(core-clp): Add `BoundedReader` to prevent out-of-bound reads in …
gibber9809 Dec 13, 2024
02d8956
build(core): Update Boost to v1.87.0 in order to pull in boost::urls;…
gibber9809 Dec 16, 2024
da66fbf
refactor(clp-s): Replace instances of `std::string const&` with `std:…
gibber9809 Dec 16, 2024
b7741c0
docs(core): Indicate dependency install scripts should be run with el…
jackluo923 Dec 16, 2024
1edc16e
feat(package)!: Add support for writing clp-s single file archives to…
haiqi96 Dec 19, 2024
e4c9dd3
fix(clp-package): Remove faulty error handling for parsing archive co…
haiqi96 Dec 19, 2024
32dc989
fix(core): Add missing `../` to fix relative header file includes. (#…
Bill-hbrhbr Dec 19, 2024
02f0b8f
Refactor lzma stream and add some doc strings.
davidlion Dec 20, 2024
fcfc73a
Fix accidental comment reflow.
davidlion Dec 20, 2024
3b89d4c
Merge branch 'main' into pr-614
davidlion Dec 20, 2024
50c8e89
Merge branch 'main' into pr-614
davidlion Dec 20, 2024
d139bd5
Apply suggestions from code review
davidlion Dec 20, 2024
68c4c36
Add missing fixes for PR suggestion.
davidlion Dec 20, 2024
dcb843e
Address review concern
Bill-hbrhbr Dec 20, 2024
b20162f
Address review concern
Bill-hbrhbr Dec 20, 2024
df41b22
nit fix
Bill-hbrhbr Dec 20, 2024
524fe1d
Change all instances of programming-error-induced error codes to Erro…
Bill-hbrhbr Dec 22, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion components/core/.clang-format
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ IncludeCategories:
# NOTE: A header is grouped by first matching regex
# Library headers. Update when adding new libraries.
# NOTE: clang-format retains leading white-space on a line in violation of the YAML spec.
- Regex: "<(absl|antlr4|archive|boost|bsoncxx|catch2|curl|date|fmt|json|log_surgeon|mongocxx\
- Regex: "<(absl|antlr4|archive|boost|bsoncxx|catch2|curl|date|fmt|json|log_surgeon|lzma|mongocxx\
|msgpack|mysql|openssl|outcome|regex_utils|simdjson|spdlog|sqlite3|string_utils|yaml-cpp|zstd)"
Priority: 3
# C system headers
Expand Down
26 changes: 24 additions & 2 deletions components/core/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,11 @@ set(CMAKE_EXPORT_COMPILE_COMMANDS ON)

# Set general compressor
set(GENERAL_COMPRESSOR "zstd" CACHE STRING "The general-purpose compressor used as the 2nd-stage compressor")
set_property(CACHE GENERAL_COMPRESSOR PROPERTY STRINGS passthrough zstd)
if ("${GENERAL_COMPRESSOR}" STREQUAL "passthrough")
set_property(CACHE GENERAL_COMPRESSOR PROPERTY STRINGS lzma passthrough zstd)
if ("${GENERAL_COMPRESSOR}" STREQUAL "lzma")
add_definitions(-DUSE_LZMA_COMPRESSION=1)
message(STATUS "Using Lempel–Ziv–Markov chain Algorithm compression")
elseif ("${GENERAL_COMPRESSOR}" STREQUAL "passthrough")
add_definitions(-DUSE_PASSTHROUGH_COMPRESSION=1)
message(STATUS "Using passthrough compression")
elseif ("${GENERAL_COMPRESSOR}" STREQUAL "zstd")
Expand Down Expand Up @@ -224,6 +227,21 @@ else()
message(FATAL_ERROR "Could not find ${CLP_LIBS_STRING} libraries for ZStd")
endif()

# Find and setup LZMA Library
# TODO: Add a script in ./cmake/Modules to properly import LZMA in find_package()'s module mode
if(CLP_USE_STATIC_LIBS)
set(LIBLZMA_USE_STATIC_LIBS ON)
endif()
find_package(LibLZMA REQUIRED)
if(LIBLZMA_FOUND)
message(STATUS "Found Lzma ${LIBLZMA_VERSION_STRING}")
message(STATUS "Lzma library location: ${LIBLZMA_LIBRARIES}")
message(STATUS "Lzma Include Dir: ${LIBLZMA_INCLUDE_DIRS}")
else()
message(FATAL_ERROR "Could not find ${CLP_LIBS_STRING} libraries for Lzma")
endif()
include_directories(${LIBLZMA_INCLUDE_DIRS})

# sqlite dependencies
set(sqlite_DYNAMIC_LIBS "dl;m;pthread")
include(cmake/Modules/FindLibraryDependencies.cmake)
Expand Down Expand Up @@ -516,6 +534,9 @@ set(SOURCE_FILES_unitTest
src/clp/streaming_compression/Compressor.hpp
src/clp/streaming_compression/Constants.hpp
src/clp/streaming_compression/Decompressor.hpp
src/clp/streaming_compression/lzma/Compressor.cpp
src/clp/streaming_compression/lzma/Compressor.hpp
src/clp/streaming_compression/lzma/Constants.hpp
Bill-hbrhbr marked this conversation as resolved.
Show resolved Hide resolved
src/clp/streaming_compression/passthrough/Compressor.cpp
src/clp/streaming_compression/passthrough/Compressor.hpp
src/clp/streaming_compression/passthrough/Decompressor.cpp
Expand Down Expand Up @@ -608,6 +629,7 @@ target_link_libraries(unitTest
clp::regex_utils
clp::string_utils
yaml-cpp::yaml-cpp
${LIBLZMA_LIBRARIES}
ZStd::ZStd
)
target_compile_features(unitTest
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
namespace clp::streaming_compression {
enum class CompressorType : uint8_t {
ZSTD = 0x10,
LZMA = 0x20,
Passthrough = 0xFF,
};
} // namespace clp::streaming_compression
Expand Down
203 changes: 203 additions & 0 deletions components/core/src/clp/streaming_compression/lzma/Compressor.cpp
Original file line number Diff line number Diff line change
@@ -0,0 +1,203 @@
#include "Compressor.hpp"

#include <array>
#include <cstddef>
#include <cstdint>
#include <cstring>

#include <lzma.h>
#include <spdlog/spdlog.h>

#include "../../ErrorCode.hpp"
#include "../../FileWriter.hpp"
#include "../../TraceableException.hpp"
#include "../../type_utils.hpp"

namespace clp::streaming_compression::lzma {
auto Compressor::open(FileWriter& file_writer) -> void {
if (nullptr != m_compressed_stream_file_writer) {
throw OperationFailed(ErrorCode_NotReady, __FILENAME__, __LINE__);
}

m_lzma_stream.detach_input();
if (false
== m_lzma_stream.attach_output(
m_compressed_stream_block_buffer.data(),
m_compressed_stream_block_buffer.size()
))
{
throw OperationFailed(ErrorCode_NoMem, __FILENAME__, __LINE__);
}
m_compressed_stream_file_writer = &file_writer;
m_uncompressed_stream_pos = 0;
}

auto Compressor::close() -> void {
if (nullptr == m_compressed_stream_file_writer) {
throw OperationFailed(ErrorCode_NotInit, __FILENAME__, __LINE__);
}

if (m_lzma_stream.avail_in() > 0) {
Bill-hbrhbr marked this conversation as resolved.
Show resolved Hide resolved
throw OperationFailed(ErrorCode_Corrupt, __FILENAME__, __LINE__);
}

flush_lzma(LZMA_FINISH);
m_lzma_stream.end_and_detach_output();
m_compressed_stream_file_writer = nullptr;
}

auto Compressor::write(char const* data, size_t data_length) -> void {
if (nullptr == m_compressed_stream_file_writer) {
throw OperationFailed(ErrorCode_NotInit, __FILENAME__, __LINE__);
}
if (false
== m_lzma_stream
.attach_input(clp::size_checked_pointer_cast<uint8_t const>(data), data_length))
{
throw OperationFailed(ErrorCode_BadParam, __FILENAME__, __LINE__);
}
encode_lzma();
m_lzma_stream.detach_input();
m_uncompressed_stream_pos += data_length;
}

auto Compressor::flush() -> void {
if (nullptr == m_compressed_stream_file_writer) {
throw OperationFailed(ErrorCode_NotInit, __FILENAME__, __LINE__);
}
flush_lzma(LZMA_SYNC_FLUSH);
}

auto Compressor::try_get_pos(size_t& pos) const -> ErrorCode {
if (nullptr == m_compressed_stream_file_writer) {
return ErrorCode_NotInit;
}
pos = m_uncompressed_stream_pos;
return ErrorCode_Success;
}

auto Compressor::encode_lzma() -> void {
while (m_lzma_stream.avail_in() > 0) {
if (0 == m_lzma_stream.avail_out()) {
flush_stream_output_block_buffer();
}
auto const rc = m_lzma_stream.lzma_code(LZMA_RUN);
switch (rc) {
case LZMA_OK:
break;
case LZMA_BUF_ERROR:
SPDLOG_ERROR("LZMA compressor input stream is corrupt. No encoding "
"progress can be made.");
throw OperationFailed(ErrorCode_Failure, __FILENAME__, __LINE__);
default:
SPDLOG_ERROR(
"lzma_code() returned an unexpected value - {}.",
static_cast<int>(rc)
);
throw OperationFailed(ErrorCode_Failure, __FILENAME__, __LINE__);
}
}
}
Bill-hbrhbr marked this conversation as resolved.
Show resolved Hide resolved

auto Compressor::flush_lzma(lzma_action flush_action) -> void {
if (false == LzmaStream::is_flush_action(flush_action)) {
SPDLOG_ERROR(
"lzma_code() supplied with invalid flush action - {}.",
static_cast<int>(flush_action)
);
throw OperationFailed(ErrorCode_BadParam, __FILENAME__, __LINE__);
}

bool flushed{false};
while (false == flushed) {
if (0 == m_lzma_stream.avail_out()) {
flush_stream_output_block_buffer();
}
auto const rc = m_lzma_stream.lzma_code(flush_action);
switch (rc) {
case LZMA_OK:
break;
case LZMA_STREAM_END:
// NOTE: flush may not have completed if a multithreaded encoder is using action
// LZMA_FULL_BARRIER. For now, we skip this check.
flushed = true;
break;
case LZMA_BUF_ERROR:
// NOTE: this can happen if we are using LZMA_FULL_FLUSH or LZMA_FULL_BARRIER. These
// two actions keeps encoding input data alongside flushing buffered encoded data.
SPDLOG_ERROR("LZMA compressor input stream is corrupt. No encoding "
"progress can be made.");
throw OperationFailed(ErrorCode_Failure, __FILENAME__, __LINE__);
Bill-hbrhbr marked this conversation as resolved.
Show resolved Hide resolved
default:
SPDLOG_ERROR(
"lzma_code() returned an unexpected value - {}.",
static_cast<int>(rc)
);
throw OperationFailed(ErrorCode_Failure, __FILENAME__, __LINE__);
}
}
flush_stream_output_block_buffer();
}

auto Compressor::flush_stream_output_block_buffer() -> void {
if (cCompressedStreamBlockBufferSize == m_lzma_stream.avail_out()) {
return;
}
m_compressed_stream_file_writer->write(
clp::size_checked_pointer_cast<char>(m_compressed_stream_block_buffer.data()),
cCompressedStreamBlockBufferSize - m_lzma_stream.avail_out()
);
if (false
== m_lzma_stream.attach_output(
m_compressed_stream_block_buffer.data(),
m_compressed_stream_block_buffer.size()
))
{
throw OperationFailed(ErrorCode_NoMem, __FILENAME__, __LINE__);
}
}

Compressor::LzmaStream::LzmaStream(int compression_level, size_t dict_size, lzma_check check) {
lzma_options_lzma options;
if (0 != lzma_lzma_preset(&options, compression_level)) {
SPDLOG_ERROR("Failed to initialize LZMA options' compression level.");
throw OperationFailed(ErrorCode_BadParam, __FILENAME__, __LINE__);
}
options.dict_size = dict_size;
std::array<lzma_filter, 2> filters{{
{.id = LZMA_FILTER_LZMA2, .options = &options},
{.id = LZMA_VLI_UNKNOWN, .options = nullptr},
}};

auto const rc = lzma_stream_encoder(&m_stream, filters.data(), check);
if (LZMA_OK == rc) {
return;
}

char const* msg{nullptr};
switch (rc) {
case LZMA_MEM_ERROR:
msg = "Memory allocation failed";
break;

case LZMA_OPTIONS_ERROR:
msg = "Specified preset is not supported";
break;

case LZMA_UNSUPPORTED_CHECK:
msg = "Specified integrity check is not supported";
break;

case LZMA_PROG_ERROR:
msg = "Input arguments are not sane";
break;

default:
msg = "Unknown error";
break;
}

SPDLOG_ERROR("Error initializing the encoder: {} (error code {})", msg, static_cast<int>(rc));
throw OperationFailed(ErrorCode_BadParam, __FILENAME__, __LINE__);
}
} // namespace clp::streaming_compression::lzma
Loading
Loading