diff options
author | shakurov <shakurov@yandex-team.com> | 2023-09-28 12:09:08 +0300 |
---|---|---|
committer | shakurov <shakurov@yandex-team.com> | 2023-09-28 12:31:24 +0300 |
commit | d89d3de523158dd97e8263955f40c68b465b0a81 (patch) | |
tree | c8b75ffe39ff890d932f1f32a519d330246e2ffe /library/python/cpp_test/conftest.py | |
parent | 9d99d203cd9fb5924a97c92caf19e3be12c4d37d (diff) | |
download | ydb-d89d3de523158dd97e8263955f40c68b465b0a81.tar.gz |
YT-20068: In TRequestQueue, fix a couple of bugs (one theoretical, one probably practical).
Theoretical: a failure in a heavy handler would lead to leaking 1) concurrency
counter and 2) the need to initiate further processing
of requests in the queue. Unlikely in practice as the only reason a heavy
handler can fail is something going wrong when deserializing a request.
Practical: ScheduleRequestsFromQueue may be reentered (because of
a synchronous Subscribe call on a throttler future). There's a latch making
a reentrant call noop. However, that would, again, lead to leaking the
need to initiate further processing of requests in the queue.
Diffstat (limited to 'library/python/cpp_test/conftest.py')
0 files changed, 0 insertions, 0 deletions