Skip to content
This repository has been archived by the owner on Oct 8, 2020. It is now read-only.

Allow for timeouts when expecting values from channels #25

Closed
cgrigis opened this issue Jan 23, 2020 · 1 comment · Fixed by #29
Closed

Allow for timeouts when expecting values from channels #25

cgrigis opened this issue Jan 23, 2020 · 1 comment · Fixed by #29
Assignees
Projects

Comments

@cgrigis
Copy link

cgrigis commented Jan 23, 2020

In many places (e.g. here, here, here, here, here, here), the code waits for information on a channel, but does not allow for a timeout. As this information is provided by other nodes over the network, the information will never come in case of crashes, and the goroutine will be stuck forever.
Allowing for a timeout in a select statement will make the code more fault-tolerant.

@mickmis mickmis added this to Backlog in MedCo Jan 27, 2020
@mickmis mickmis moved this from Backlog to To Do v0.3.0 in MedCo Jan 27, 2020
@JoaoAndreSa
Copy link
Contributor

pull request #29 should address/fix this issue

@mickmis mickmis linked a pull request Feb 7, 2020 that will close this issue
@mickmis mickmis closed this as completed Feb 10, 2020
MedCo automation moved this from To Do v0.3.0 to Done Feb 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
No open projects
MedCo
  
Done
Development

Successfully merging a pull request may close this issue.

3 participants