ydb-platform / nbs

Network Block Store
Apache License 2.0
50 stars 14 forks source link

Replace SessionManager by SessionFactory #1477

Open sanek325 opened 1 week ago

sanek325 commented 1 week ago

В SessionManager не надо хранить список открытых сессий, потому что с SessionManager работает только EndpointManager, который уже хранит список открытых ендпоинтов с их сессиями. Таким образом из SessionManager можно вместе со списком сессий удалить все проверки на то что сессия уже открыта и т.д. Все эти проверки уже есть в EndpointManager. Таким образом это будет уже не SessionManager, а SessionFactory.

github-actions[bot] commented 1 week ago

[!NOTE] This is an automated comment that will be appended during run.

:red_circle: linux-x86_64-relwithdebinfo: some tests FAILED for commit e7af94a224bcb3d7b5db014788473319fb45f1a6. TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5288 5284 0 1 1 2