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.

Sign up for GitHub

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

Jump to bottom

Single supervisor for multi path relayer #862

Closed
5 tasks
ancazamfir opened this issue Apr 28, 2021 · 0 comments · Fixed by #881
Closed
5 tasks

Single supervisor for multi path relayer #862

ancazamfir opened this issue Apr 28, 2021 · 0 comments · Fixed by #881
Assignees
@romac @adizere
Labels
E: gravity External: related to Gravity DEX I: logic Internal: related to the relaying logic O: new-feature Objective: cause to add a new feature or support
Milestone
04.2021

Comments

@ancazamfir
Copy link
Collaborator

ancazamfir commented Apr 28, 2021 •
edited by adizere
Loading

Crate

relayer

Summary

Use a single supervisor per relayer instance instead of one per connection.

Also simplifies the config content (remove connections & paths sections) (bundle with #867).
Also should simplify a REST API integration.

Problem Definition

This was the original intent to start with. Also, it will avoid multiple subscriptions and propagation for identical events.
I think it will also be easier when interacting with the rest server in the future, maintaining configuration, caching state, etc.

Proposal

Acceptance Criteria

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@ancazamfir ancazamfir added this to the 04.2021 milestone Apr 28, 2021
@romac romac mentioned this issue Apr 28, 2021
Closed
11 tasks
@romac romac added I: logic Internal: related to the relaying logic O: new-feature Objective: cause to add a new feature or support in progress labels Apr 28, 2021
@romac romac added the E: gravity External: related to Gravity DEX label Apr 29, 2021
@romac romac mentioned this issue Apr 30, 2021
Merged
10 tasks
@ancazamfir ancazamfir mentioned this issue May 6, 2021
Merged
26 tasks
@ancazamfir ancazamfir mentioned this issue May 7, 2021
Closed
5 tasks
This was referenced Jan 19, 2022
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@romac romac

@adizere adizere

Labels
E: gravity External: related to Gravity DEX I: logic Internal: related to the relaying logic O: new-feature Objective: cause to add a new feature or support
Projects
None yet
Milestone
04.2021
Development

Successfully merging a pull request may close this issue.

Use a single supervisor to subscribe to all configured chains informalsystems/hermes
3 participants
@romac @adizere @ancazamfir