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

Process messages from the same chain concurrently #31

Closed
Tracked by #242
cam-schultz opened this issue Sep 6, 2023 · 0 comments · Fixed by #288
Closed
Tracked by #242

Process messages from the same chain concurrently #31

cam-schultz opened this issue Sep 6, 2023 · 0 comments · Fixed by #288
Assignees
Labels
enhancement New feature or request idea Consider as a possible enhancement

Comments

@cam-schultz
Copy link
Collaborator

Context and scope
When adding bootstrapping logic and state in #20, we identified issues with processing messages from the same source chain concurrently. The workaround was to process such messages in serial in order to provide stronger guarantees of block ordering within the application and database.

Discussion and alternatives
We should consider adding back in this concurrent processing of messages from the same source chain in order to potentially improve performance.

Open questions
Under what conditions will adding this concurrency improve performance? In any case, relayer message processing is capped by the rate at which messages are included in the source chain, so we should make sure this would yield performance improvements before pursuing, especially since the synchronization logic is likely to be complex.

@cam-schultz cam-schultz added enhancement New feature or request idea Consider as a possible enhancement labels Sep 6, 2023
@cam-schultz cam-schultz added this to the Post-Durango fast follows milestone Mar 8, 2024
@cam-schultz cam-schultz mentioned this issue Apr 18, 2024
@cam-schultz cam-schultz self-assigned this May 17, 2024
@cam-schultz cam-schultz moved this from Backlog 🗄️ to In Progress 🏗 in Platform Engineering Group May 17, 2024
@github-project-automation github-project-automation bot moved this from In Progress 🏗 to Done ✅ in Platform Engineering Group Jun 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request idea Consider as a possible enhancement
Projects
Archived in project
Status: 📋 Backlog
Development

Successfully merging a pull request may close this issue.

1 participant