ydb/library/cpp/yson/node/node_visitor.cpp, branch oidc-1.2.5-devMirror of YDB github reposhttps://code.mastervirt.ru/ydb/atom?h=oidc-1.2.5-dev2024-08-13T07:23:50Zremove unused include2024-08-13T07:23:50Ztobotobo@yandex-team.com2024-08-13T07:13:55Zurn:sha1:7330529872e88a3f5195fde70427196ce44e5924
637fa63ff4633dd45416c48e111f38493cc00cf0
Y_FAIL->Y_ABORT at '^li'2023-10-17T06:00:07Zilnurkhilnurkh@yandex-team.com2023-10-17T05:15:44Zurn:sha1:784925324fd115c37bc98c5bbfe64c15f9966d74
https://clubs.at.yandex-team.ru/arcadia/29404
Avoid unnecessary copies when visiting yson graph2022-08-23T10:35:29Zzhizhinzhizhin@yandex-team.com2022-08-23T10:35:29Zurn:sha1:e200c8e2aa45d15f672a8ea2029e39a9f1620e6a
При вызове лямбды происходит деаллокация большого числа TNode. Они деаллоцируются внутри
лямбд так как лямбды принимают `const std::pair<TString, TNode>&`, а
`THashMap` итерирует по `const std::pair<const TString, TNode>&`
Меняю явный тип на `const auto&`.
Предыстория:
После перехода на TC Malloc начали в сервере ловить странные кордампы
при десериализации YSON в строку.
Пример дампа: https://coredumps.n.yandex-team.ru/core_trace?core_id=47595674
Вот как вызывали: https://a.yandex-team.ru/arcadia/voicetech/library/asr_logs/message_logger.cpp?rev=rXXXXXX#L161
Вычитка кода показала, что переменная EngineInfo всегда валидная (ссылка
на константную глобальную переменную, которая жива всегда во время
работы программы).
В кордампе заметили деаллокации внутри этих лямбд и поняли, что происходят ненужные копирования.
Предполагаем, что деаллокации внутри YSON могут быть причиной кордампов.Restoring authorship annotation for <ermolovd@yandex-team.ru>. Commit 2 of 2.2022-02-10T13:47:53Zermolovdermolovd@yandex-team.ru2022-02-10T13:47:53Zurn:sha1:2a4a975b112fa0fa138abc7457fe67e0e1e7fd02Restoring authorship annotation for <ermolovd@yandex-team.ru>. Commit 1 of 2.2022-02-10T13:47:53Zermolovdermolovd@yandex-team.ru2022-02-10T13:47:53Zurn:sha1:b91bcef7c99a119b0846479ef71bfab4beec72caRestoring authorship annotation for <levysotsky@yandex-team.ru>. Commit 2 of 2.2022-02-10T13:47:29Zlevysotskylevysotsky@yandex-team.ru2022-02-10T13:47:29Zurn:sha1:32b231c8474a1ade4bdf776ade6a20341691d9d7Restoring authorship annotation for <levysotsky@yandex-team.ru>. Commit 1 of 2.2022-02-10T13:47:29Zlevysotskylevysotsky@yandex-team.ru2022-02-10T13:47:29Zurn:sha1:57f874ffc2a75047c1c4fea7a9fc86cb0f56ed50Restoring authorship annotation for <monster@yandex-team.ru>. Commit 2 of 2.2022-02-10T13:47:19Zmonstermonster@yandex-team.ru2022-02-10T13:47:19Zurn:sha1:b23c1d7a8015c2006a148fd93b84cdeb0aee17a3Restoring authorship annotation for <monster@yandex-team.ru>. Commit 1 of 2.2022-02-10T13:47:19Zmonstermonster@yandex-team.ru2022-02-10T13:47:19Zurn:sha1:dd76ae1f6213d065375ab296699f764faafbe5bdRestoring authorship annotation for Vlad Yaroslavlev <vladon@vladon.com>. Commit 2 of 2.2022-02-10T13:46:25ZVlad Yaroslavlevvladon@vladon.com2022-02-10T13:46:25Zurn:sha1:344ea37b4a345701ab0e67de2266a1c1bd7baf2d