Opened 8 years ago

Closed 8 years ago

Last modified 6 years ago

#180 closed улучшение (fixed)

Устранить ненужные запросы списка пользователей

Reported by: alx Owned by: dimag
Priority: minor Milestone: 2 очередь
Component: ПО MC04-Dispatcher. Пульт диспетчера/техника Keywords:
Cc: san

Description

Сейчас программа ведет себя неоптимально в плане запроса списка пользователей (list_users) с коммутатора. Так, список пользователей запрашивается у коммутатора каждый раз при нажатии пользователем ConnectButton, а также каждый раз при регистрации/разрегистрации или протухании регистрации любого пользователя коммутатора.

Такая логика неоптимальна, и подобное поведение создает ненужную лишнюю нагрузку как на коммутатор, так и на саму программу. Так, если на коммутаторе будет несколько тысяч пользователей, то даже при периоде регистрации каждого 1 час (3600 секунд) сообщение о регистрации (и, соответственно, перезапрос всего списка) будет приходить каждую секунду.

Список пользователей должен запрашиваться один раз при старте программы и при выполнении на коммутаторе reloadxml. ИМХО без выполнения reloadxml список пользователей измениться не может, и, следовательно, перезапрашивать его нет смысла.

Change History (5)

comment:1 by san, 8 years ago

Поправка: при старте программы(или переподключении к сокету) и при выполнении на коммутаторе reloadxml.

comment:2 by dimag, 8 years ago

Resolution: fixed
Status: newclosed

rev. 227

comment:3 by alx, 8 years ago

Ненужные list_users из кода пропали, зато теперь я вижу, что при регистрации/разрегистрации выполняется list_users user ..., результат которого затем передается в currentConfig.RefreshUserState(), в связи с чем возник вопрос о цели данного запроса. Какая именно информация о пользователе и зачем здесь обновляется?

comment:4 by alx, 8 years ago

Так как в результате исправления #169 в r234 запросы list_users исчезли, вопрос снимается. :)

comment:5 by san, 6 years ago

Milestone: Текущее2 очередь

Milestone renamed

Note: See TracTickets for help on using tickets.