Issue #90 Remove zstd06 from contrib

[tsayukov]

Issue #90

Способов получить нужный кодек не так много, это функции:

• NBlockCodecs::Codec, передаём в неё название кодека, например "zstd06_level" (level --- уровень сжатия);
• NBlockCodecs::ListAllCodecs и NBlockCodecs::ListAllCodecsAsString, чтобы получить все зарегистрированные кодеки. Регистрация кодеков происходит в анонимном неймспейсе при инициализации глобальной переменной. Даже если заинклюдить соответствующий cpp-файл, прямого доступа получить не получится, разве что запустить заново регистрацию, но уже существующий кодек ещё раз не добавиться (т.к. хранилище --- хэш-мапа).

В библиотеке cpp-http-io (library/cpp/http/io/compression.cpp:42) происходит добавление существующих кодеков в собственную кодек-фактори, но кодеки zstd08 (он же zstd) и zstd06 (искомый) не добавляются. Других использований ListAllCodecs* вне library/cpp/blockcodecs не обнаружено. Проверяя, передавали ли в метод NBlockCodecs::Codec строку, начинающуюся на "zstd06" (других алиасов у кодека нет), выяснено, что происходило это только в юнит-тестах, чтобы проверить факт регистрации всех возможных кодеков.

Т.к. таргет blockcodecs-codecs-legacy_zstd06 в blockcodecs линкуют публично, его в теории могли бы использовать в остальных зависимых библиотеках, но нужных хедеров из contrib/libs/zstd06 никто не включал.

Более общий поиск по "zstd" показал, что в library есть библиотеки stream и monlib, но они используют zstd напрямую, линкуясь к ZSTD::ZSTD (то есть это не наша искомая zstd06). Ещё следы zstd нашлись в client, но там та же самая картина: либо это зависимость от cpp-stream-zstd, либо от сpp-monlib-*.

Таким образом, можно с большой долей уверенности сказать, что от использования zstd06 можно безопасно отказаться.

[GitSparTV]

Привет. В коде только удаления, так и должно быть? Замен не требуется?

[tsayukov]

Привет. В коде только удаления, так и должно быть? Замен не требуется?

@GitSparTV Да, было предложено отказаться от этой либы, если она нигде не используется!