search

docker / go-events

Composable event distribution for Go
Apache License 2.0
132 stars 23 forks source link

retry: use time.After rather than manage timer lifecycle #8

Closed stevvooe closed 8 years ago

stevvooe commented 8 years ago

After some analysis of stdlib, we found that time.After is sufficient to avoid leaking a goroutine. This is because the send on the time channel is actaully non-blocking.

Signed-off-by: Stephen J Day stephen.day@docker.com

cc @aaronlehmann

aaronlehmann commented 8 years ago

LGTM

Would merge this if I had write access.

stevvooe commented 8 years ago

@aaronlehmann Flex your new found power!