matryer / vice

Go channels at horizontal scale (powered by message queues)
https://medium.com/@matryer/introducing-vice-go-channels-across-many-machines-bcac1147d7e2
Apache License 2.0
1.54k stars 79 forks source link

redis: reuse connection and empty buffer on Stop #26

Closed HeavyHorst closed 7 years ago

HeavyHorst commented 7 years ago

http://godoc.org/github.com/go-redis/redis#Client says

Client is a Redis client representing a pool of zero or more underlying connections. It's safe for concurrent use by multiple goroutines.

It should be no problem to reuse this Client.

This PR fixes also #24

HeavyHorst commented 7 years ago

don't merge - data race

HeavyHorst commented 7 years ago

@matryer i hope i got it, all travis runs are green.

matryer commented 7 years ago

Nice on @HeavyHorst.