search

BedrockStreaming / Statsd

PHP component easing the statsd usage
13 stars 14 forks source link

Add count, set and gauge possibility #2

Closed jdeniau closed 10 years ago

jdeniau commented 10 years ago

The increment function was nice, but the basic count was missing (how do you log website memory btw ?)

I added the gauge and set possibility too.

omansour commented 10 years ago

txs ! i'll check that

we use a custom timer to log memory : https://github.com/M6Web/StatsdBundle/blob/master/doc/usage.md#timers (not sure if it's the best way but it is how we do)

jdeniau commented 10 years ago

I'm working on a bundle to interact with the stack. I will let you know as soon as it is ready Le 14 mars 2014 20:30, "Olivier Mansour" notifications@github.com a écrit :

txs ! i'll check that

we use a custom timer to log memory : https://github.com/M6Web/StatsdBundle/blob/master/doc/usage.md#timers(not sure if it's the best way but it is how we do)

Reply to this email directly or view it on GitHubhttps://github.com/M6Web/Statsd/pull/2#issuecomment-37686373 .

omansour commented 10 years ago

:+1: sounds ok but maybe we should write more tests on thoses methods

KuiKui commented 10 years ago

Yes, these tests does not validate behaviors but only returned values.