ydb/library/cpp, branch CLI_2.17.0Mirror of YDB github reposhttps://code.mastervirt.ru/ydb/atom?h=CLI_2.17.02024-12-03T08:17:42ZMerge branch 'rightlib' into mergelibs-241203-08162024-12-03T08:17:42ZAlexander Smirnovalex@ydb.tech2024-12-03T08:17:42Zurn:sha1:003b31a5bdd9d6044c663395cd47c8ae02bc0bb3Intermediate changes2024-12-03T03:30:41Zrobot-pigletrobot-piglet@yandex-team.com2024-12-03T03:18:16Zurn:sha1:9238ddd7f5289182911ffdddefc03d68e6a98cf3
commit_hash:6f0d83406a245425d156e961c3519d5bd822857a
Add missing configs + add mold and coverage to piglet YDB config2024-12-02T16:31:00Zmaxim-yurchukmaxim-yurchuk@yandex-team.com2024-12-02T16:15:43Zurn:sha1:e4293e758af131e7ef7ca74bfa1ef6f9b74b58cb
commit_hash:c0112c0409a0972eb859b32a014980020df7ce0a
Merge branch 'rightlib' into mergelibs-241202-14532024-12-02T14:55:03ZAlexander Smirnovalex@ydb.tech2024-12-02T14:55:03Zurn:sha1:3301787707269da1e7b44e909ce54db3b40adcadNew version of the tld SKIP_CHECK SKIP_REVIEW2024-12-01T23:15:24Zrobot-ratatoskrobot-ratatosk@yandex-team.com2024-12-01T23:04:08Zurn:sha1:862a269ad6191fa7590f00e58cfca2340184e625
commit_hash:fac37a51fb5b0fdd660beccb43a2a8861308e39f
Intermediate changes2024-11-30T21:09:15Zrobot-pigletrobot-piglet@yandex-team.com2024-11-30T20:55:24Zurn:sha1:22f8e802732231ee090cd47a053afd31b32e06bf
commit_hash:320ab736dea5dae74e2ec8c344fb915be4c4df99
Merge branch 'rightlib' into mergelibs-241130-18152024-11-30T18:17:04ZAlexander Smirnovalex@ydb.tech2024-11-30T18:17:04Zurn:sha1:6d314ad5bbedbdbad47e567f12669f005eaad866YT-21233: Introduce ToAttributeValue cpo which removes dependency of TErrorAttribute from yt/core2024-11-29T19:12:26Zarkady-e1ppaarkady-e1ppa@yandex-team.com2024-11-29T19:02:21Zurn:sha1:b21317884da9622ac16bbad26627ae014d72d6a7
Plan:
1) Remove `IAttributedDictionary` type from the public API. \+
2) Remove `Set` method from public API in favor of `operator<<=`. \+
3) Adopt `ConvertTo<T>` (or other name) CPO with proper extension into `NYT::NYson::ConvertTo` from `yt/core`.
4) Use CPO from (3) to eliminate direct dependency on `yt/core` of `Get/Find` methods from attributes API.
5) Adopt `ConvertToYsonString` (or other name) CPO with proper extension into `yt/core` customisations.
6) Use CPO from (5) to eliminate direct dependency on `yt/core` of `TErrorAttribute` ctor.
7) Swap attributes implementation to the one which doesn’t use `IAttributeDictionary`.
8) At this point `stripped_error*` can be moved to library/cpp/yt and so can recursively dependant on THROW macro methods `Get/Find/…`.
9) Adjust CPO’s to work with `std::string` instead of `TYsonString` assuming text format to be used (maybe `TString` for now).
10) Remove dep of `library/cpp/yt/error` on `yson` entirely.
This pr addresses 5-6 steps of plan. Below is a brief explanation of design decisions. We also expressed everything related to key-value code in terms of aliases of `TErrorAttribute` so that later we could make a relatively simple switch to `std::string`. We expect to do steps 7-8 in the next pull request as everything should be ready
commit_hash:de9feca2bd24d823b33d904ef0fa5f4856f9b020
Intermediate changes2024-11-29T15:40:16Zrobot-pigletrobot-piglet@yandex-team.com2024-11-29T15:15:29Zurn:sha1:2aee7cda6c758679145563340ff94fc652670ea1
commit_hash:60e005cdf76d5bff2a370a6b8f35ef4f6792f414
Merge branch 'rightlib' into mergelibs-241129-13302024-11-29T13:32:04ZAlexander Smirnovalex@ydb.tech2024-11-29T13:32:04Zurn:sha1:03fc6a84cd47911dfcc873669a69da6cbcc308d1