aboutsummaryrefslogtreecommitdiffstats
path: root/library/cpp/blockcodecs/codecs
diff options
context:
space:
mode:
authorprime <prime@yandex-team.com>2022-08-30 10:49:22 +0300
committerprime <prime@yandex-team.com>2022-08-30 10:49:22 +0300
commit2844ceec0b5580e1f1d27a0542a278e9df7ea5a3 (patch)
tree124b0688167813f6e9610eaf4143c0ab85f77374 /library/cpp/blockcodecs/codecs
parent329b8e356a6d854f393ebb4704a15a1b762cb656 (diff)
downloadydb-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.txt2
-rw-r--r--library/cpp/blockcodecs/codecs/lz4/lz4.cpp16
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>>());