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

Remove bottleneck on chain runtime for sending queries concurrently #3344

Open
5 tasks
romac opened this issue May 16, 2023 · 0 comments
Open
5 tasks

Remove bottleneck on chain runtime for sending queries concurrently #3344

romac opened this issue May 16, 2023 · 0 comments
Assignees
Labels
A: risky Admin: cause high impact risk O: performance Objective: cause to improve performance
Milestone

Comments

@romac
Copy link
Member

romac commented May 16, 2023

Summary

By removing the bottleneck on the chain runtime, we can issue queries concurrently rather than serially as we do now, which has the potential to speed up relaying in two cases:

a) when relaying on multiple channels between two or more chains
b) when relaying between a chain A and two or more chains B, C, ... with A<->B, A<->C, ...

Note that it won't let us send multiple txs concurrently because of the monotonic constraint on the account sequence number. To speed up this aspect of the relayer, we would need multiple wallets.


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@romac romac added the O: performance Objective: cause to improve performance label May 16, 2023
@romac romac self-assigned this May 16, 2023
@github-project-automation github-project-automation bot moved this to 🩹 Triage in Hermes May 16, 2023
@romac romac mentioned this issue May 16, 2023
7 tasks
@seanchen1991 seanchen1991 moved this from 🩹 Triage to 🏗 In progress in Hermes May 16, 2023
@seanchen1991 seanchen1991 added this to the v1.6 milestone May 16, 2023
@seanchen1991 seanchen1991 moved this from 🏗 In progress to 🔖 Needs review in Hermes May 17, 2023
@seanchen1991 seanchen1991 moved this from 🔖 Needs review to 👀 In review in Hermes May 17, 2023
@romac romac modified the milestones: v1.6, v1.7 Jul 18, 2023
@romac romac modified the milestones: v1.7, v1.8 Aug 23, 2023
@romac romac modified the milestones: v1.8, v1.9 Nov 9, 2023
@romac romac modified the milestones: v1.9, v2 Mar 6, 2024
@romac romac added the A: risky Admin: cause high impact risk label Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: risky Admin: cause high impact risk O: performance Objective: cause to improve performance
Projects
Status: 🔖 In review
Development

Successfully merging a pull request may close this issue.

2 participants