ydb/library/cpp/yson/node/node_visitor.cpp, branch get_cmakelists2Mirror of YDB github reposhttps://code.mastervirt.ru/ydb/atom?h=get_cmakelists22022-08-23T10:35:29ZAvoid 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:344ea37b4a345701ab0e67de2266a1c1bd7baf2dRestoring authorship annotation for Vlad Yaroslavlev <vladon@vladon.com>. Commit 1 of 2.2022-02-10T13:46:23ZVlad Yaroslavlevvladon@vladon.com2022-02-10T13:46:23Zurn:sha1:706b83ed7de5a473436620367af31fc0ceecde07intermediate changes2022-02-07T15:08:42ZDevtools Arcadiaarcadia-devtools@yandex-team.ru2022-02-07T15:08:42Zurn:sha1:1110808a9d39d4b808aef724c861a2e1a38d2a69
ref:cde9a383711a11544ce7e107a78147fb96cc4029