diff options
| author | prime <[email protected]> | 2022-08-30 10:49:22 +0300 | 
|---|---|---|
| committer | prime <[email protected]> | 2022-08-30 10:49:22 +0300 | 
| commit | 2844ceec0b5580e1f1d27a0542a278e9df7ea5a3 (patch) | |
| tree | 124b0688167813f6e9610eaf4143c0ab85f77374 /library/cpp/blockcodecs | |
| parent | 329b8e356a6d854f393ebb4704a15a1b762cb656 (diff) | |
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')
| -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 299fcbd9b20..3f0a426090f 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 fbf0fe110f1..94fcf4dd969 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>>());  | 
