search

alexcesaro / statsd

An efficient Statsd Go client.
MIT License
301 stars 79 forks source link

don't mute connection when listener is absent #8

Open christian-blades-cb opened 8 years ago

christian-blades-cb commented 8 years ago

Corrected tests for #7 to account for expected new behavior.

Fixes #6

christian-blades-cb commented 8 years ago

@alexcesaro @zstyblik Hopefully a version with passing tests is more palatable. Or at least can spark a conversation.

I also would like to remove launch order dependencies. Having the stats emitted and available when the collector finally comes up is the behavior I would expect.

zstyblik commented 8 years ago

@christian-blades-cb, cool :)

vchernoy commented 7 years ago

Any estimation when to expect the PR to be merged? This functionality is desired. Thank you.

christian-blades-cb commented 7 years ago

@vchernoy Given that @alexcesaro hasn't pushed any commits since April, it's probably time to either fork this repo or move on to a different one. I chose cactus/go-statsd-client for my recent production work.