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

feat: check out tokio channels #54

Closed
erhant opened this issue Jun 24, 2024 · 1 comment
Closed

feat: check out tokio channels #54

erhant opened this issue Jun 24, 2024 · 1 comment
Assignees
Labels
refactor This needs refactoring

Comments

@erhant
Copy link
Member

erhant commented Jun 24, 2024

We may make use of channels for various reasons:

  • Use a oneshot channel for cancellation, instead of tokens maybe
  • More importantly, use channels to sync between threads such as for P2P and Node.

Relevant links:

@erhant erhant added the refactor This needs refactoring label Jul 3, 2024
@erhant erhant changed the title feat: use onehsot channel instead of cloned Cancellation tokens? feat: check out tokio channels Aug 29, 2024
@erhant erhant self-assigned this Aug 29, 2024
@erhant
Copy link
Member Author

erhant commented Sep 27, 2024

Cancellation works well, closing this.

@erhant erhant closed this as completed Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor This needs refactoring
Projects
None yet
Development

No branches or pull requests

1 participant