Dv
Хм, думаешь он сначала банит, а потом разбанивает сам же?
Допускаю, что так.
KickChatMemberAsync работает дольше, чем RestrictChatMemberAsyncSize: a a a
2020 October 11
Dv
Особенно если
Dv
Ладно, я вижу несколько решений
Dv
Во-первых, можно сделать оптимистичную проверку перед следующим действием
Dv
А не удалили ли ещё юзера, которого мы хотим разбанить, из коллекции пользователей (что будет означать. что его уже отпроцессили в другом потоке)
Dv
Но это всё ещё не супер надёжно
Dv
Можно просто юзера пораньше удалять.
Dv
Ну, типа, взяли и сразу удалили.
GetAndUpdate там наверняка какой-нибудь есть.AB
Хм, read-write slim ещё в голову приходит.
Dv
Хм, read-write slim ещё в голову приходит.
Смотря как ты его будешь шардировать.
AB
Смотря как ты его будешь шардировать.
А вот это вопрос хороший.
Dv
Если пошардить блокировки по юзерам — то будет норм
Dv
Но тогда будет довольно дохрена блокировок
c
Minoru
насколько я вижу, _usersStore используется только для принятия решения в первом if. Поэтому, во-первых, можно Get и Remove переместить в критическую секцию в начале кода, во-вторых, наверняка есть какой-нибудь метод Take, возвращающий Option — лучше переехать на него
насколько я вижу, _usersStore используется только для принятия решения в первом if. Поэтому, во-первых, можно Get и Remove переместить в критическую секцию в начале кода, во-вторых, наверняка есть какой-нибудь метод Take, возвращающий Option — лучше переехать на него
Dv
Там под капотом есть
TryGetValue, вот типа тогоc
Minoru
нам надо не просто Get, нам надо, чтобы элемент был удалён из коллекции
нам надо не просто Get, нам надо, чтобы элемент был удалён из коллекции
Dv
Ага, и получается, что конкурентная коллекция не нужна, и сгодится обычная с блокировкой.
Dv
Правда, там есть ещё какой-то
GetAll непонятныйDv
Который используется в методе
BanSlowUsers %)c
Minoru
ну, надо смотреть на этот UserStore, может, там ещё какие-то методы есть, для которых нужна конкурентность. Но в рамках вот этой вот функции — да, я считаю, что конкурентность не нужна
ну, надо смотреть на этот UserStore, может, там ещё какие-то методы есть, для которых нужна конкурентность. Но в рамках вот этой вот функции — да, я считаю, что конкурентность не нужна