Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Investigate whether running concurrent elections instances under a single account may cause throttling #58

Open
samliew opened this issue Aug 17, 2021 · 1 comment
Assignees
Labels
question Further information is requested

Comments

@samliew
Copy link
Owner

samliew commented Aug 17, 2021

Should this bot be able to run in multiple election rooms, will this cause an issue if it's all on the same Stack Exchange chat account?

Should we need to create mutiple accounts for the bot to utilise in chat?

If possible we should be able to run everything via a single account, which means keeping track of when the last message was sent and then back-off.

@samliew samliew added the question Further information is requested label Aug 17, 2021
@Oaphi
Copy link
Collaborator

Oaphi commented Aug 17, 2021

Hmm, that's a good question. We do keep track of lastMessageTime for the purposes of throttling, so this should not be hard to extend to multiple elections. One caveat, though, is that we might be ending up with significant delays in responses for some of the rooms should there be multiple elections happening at once.

That said, I think a single-account bot that can work with multiple elections at the same time is possible if we switch from dropping messages that come in during the throttle period (current behaviour) in favour of a message queue processed in order regardless of the initiator room with maybe some load balancing to avoid ending up with a LILO queue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants