It looked like a tcp entry point at first glance, hence my comment. However I do think it belongs in the test folder. Even tough there's no harm in having it where it is, it makes it clear that it's not part of the running system.


Sent from Samsung Mobile


-------- Original message --------
From: Ignas Vyšniauskas
Date:19/02/2014 23:29 (GMT+01:00)
To: spilgames/estatsd
Cc: Dennis Docter
Subject: Re: [estatsd] Cleanup and testing (#6)


In src/estatsd_receiver.erl:

> @@ -0,0 +1,91 @@
> +%@doc Helper module to receive stats from Graphite.


I think the receiver is great. lager e.g. has multiple backends<https://github.com/basho/lager/tree/master/test> just for testing, it's very convenient and makes it easy to extend with new functionality while keeping it testable.

—
Reply to this email directly or view it on GitHub<https://github.com/spilgames/estatsd/pull/6/files#r9886700>.