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

Feature Request: concurrent update handling in dispatching2 #536

Closed
Hirrolot opened this issue Mar 3, 2022 · 2 comments
Closed

Feature Request: concurrent update handling in dispatching2 #536

Hirrolot opened this issue Mar 3, 2022 · 2 comments
Assignees
Labels
feature-accepted Feature Request or Proposal was accepted by maintainers and will be implemented K-feature-request Kind: request for implementing a feature
Milestone

Comments

@Hirrolot
Copy link
Collaborator

Hirrolot commented Mar 3, 2022

Apparently, it turns out that dispatching2 handles all updates sequentially. This is undesired for obvious reasons. I suggest making it concurrent to allow proper scaling with a growing number of bot users.

@Hirrolot Hirrolot added the K-feature-request Kind: request for implementing a feature label Mar 3, 2022
@Hirrolot Hirrolot self-assigned this Mar 3, 2022
@WaffleLapkin WaffleLapkin added the feature-accepted Feature Request or Proposal was accepted by maintainers and will be implemented label Mar 3, 2022
@Hirrolot Hirrolot added this to the v0.8.0 milestone Mar 24, 2022
@Hirrolot
Copy link
Collaborator Author

Also, don't forget that updates from the same chats should be handled sequentially, one by one. We can use something like flurry to associate chat IDs to Tokio channels handling updates.

@WaffleLapkin
Copy link
Contributor

Implemented by #585 and #589

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-accepted Feature Request or Proposal was accepted by maintainers and will be implemented K-feature-request Kind: request for implementing a feature
Projects
None yet
Development

No branches or pull requests

2 participants