diff options
author | prime <prime@yandex-team.com> | 2022-08-30 10:49:22 +0300 |
---|---|---|
committer | prime <prime@yandex-team.com> | 2022-08-30 10:49:22 +0300 |
commit | 2844ceec0b5580e1f1d27a0542a278e9df7ea5a3 (patch) | |
tree | 124b0688167813f6e9610eaf4143c0ab85f77374 /library/cpp/blockcodecs/codecs | |
parent | 329b8e356a6d854f393ebb4704a15a1b762cb656 (diff) | |
download | ydb-2844ceec0b5580e1f1d27a0542a278e9df7ea5a3.tar.gz |
Remove lz4 build with custom memory configuration
lz4 позволяет определить параметр memory на этапе компиляции. Этот параметр задаёт трейдофф между скоростью и степенью сжатия.
Сейчас этот параметр используется только в blockcodecs. Поиск по аркадии не находит упоминания этих кодеков. Возможно, что они вообще не используются.
https://a.yandex-team.ru/search?search=lz4-fast%5Cd%5Cd-fast,,,arcadia,,500&repo=arcadia
https://a.yandex-team.ru/search?search=lz4-fast%5Cd%5Cd-safe,,,arcadia,,500&repo=arcadia
Так как параметр на этапе компиляции требует хаков на уровне сборки и усложняет апдейт lz4, предлагаю удалить эти хаки, и оставить только default уровень сжатия (параметр 14).
Diffstat (limited to 'library/cpp/blockcodecs/codecs')
-rw-r--r-- | library/cpp/blockcodecs/codecs/lz4/CMakeLists.txt | 2 | ||||
-rw-r--r-- | library/cpp/blockcodecs/codecs/lz4/lz4.cpp | 16 |
2 files changed, 4 insertions, 14 deletions
diff --git a/library/cpp/blockcodecs/codecs/lz4/CMakeLists.txt b/library/cpp/blockcodecs/codecs/lz4/CMakeLists.txt index 299fcbd9b2..3f0a426090 100644 --- a/library/cpp/blockcodecs/codecs/lz4/CMakeLists.txt +++ b/library/cpp/blockcodecs/codecs/lz4/CMakeLists.txt @@ -12,7 +12,6 @@ target_link_libraries(blockcodecs-codecs-lz4 INTERFACE contrib-libs-cxxsupp yutil contrib-libs-lz4 - libs-lz4-generated cpp-blockcodecs-core ) @@ -21,7 +20,6 @@ target_link_libraries(blockcodecs-codecs-lz4.global PUBLIC contrib-libs-cxxsupp yutil contrib-libs-lz4 - libs-lz4-generated cpp-blockcodecs-core ) target_sources(blockcodecs-codecs-lz4.global PRIVATE diff --git a/library/cpp/blockcodecs/codecs/lz4/lz4.cpp b/library/cpp/blockcodecs/codecs/lz4/lz4.cpp index fbf0fe110f..94fcf4dd96 100644 --- a/library/cpp/blockcodecs/codecs/lz4/lz4.cpp +++ b/library/cpp/blockcodecs/codecs/lz4/lz4.cpp @@ -4,7 +4,6 @@ #include <contrib/libs/lz4/lz4.h> #include <contrib/libs/lz4/lz4hc.h> -#include <contrib/libs/lz4/generated/iface.h> using namespace NBlockCodecs; @@ -18,12 +17,11 @@ namespace { struct TLz4FastCompress { inline TLz4FastCompress(int memory) : Memory(memory) - , Methods(LZ4Methods(Memory)) { } inline size_t DoCompress(const TData& in, void* buf) const { - return Methods->LZ4CompressLimited(in.data(), (char*)buf, in.size(), LZ4_compressBound(in.size())); + return LZ4_compress_default(in.data(), (char*)buf, in.size(), LZ4_compressBound(in.size())); } inline TString CPrefix() { @@ -31,7 +29,6 @@ namespace { } const int Memory; - const TLZ4Methods* Methods; }; struct TLz4BestCompress { @@ -93,20 +90,15 @@ namespace { struct TLz4Registrar { TLz4Registrar() { - for (int i = 0; i < 30; ++i) { + for (int i = 10; i <= 20; ++i) { typedef TLz4Codec<TLz4FastCompress, TLz4FastDecompress> T1; typedef TLz4Codec<TLz4FastCompress, TLz4SafeDecompress> T2; THolder<T1> t1(new T1(i)); THolder<T2> t2(new T2(i)); - if (t1->Methods) { - RegisterCodec(std::move(t1)); - } - - if (t2->Methods) { - RegisterCodec(std::move(t2)); - } + RegisterCodec(std::move(t1)); + RegisterCodec(std::move(t2)); } RegisterCodec(MakeHolder<TLz4Codec<TLz4BestCompress, TLz4FastDecompress>>()); |