lynx

lynx copied to clipboard

Hey there, Thanks for this great library, it's great to use so far 🎉 Just wanted to make the readme a little more readable :)

DanielMSchmidt

I'm locally running a web service that sends a timing metric (for each incoming request) to statsd (using this lynx client library), which then sends the metrics to carbon relay,...

edwmurph

With this change it will be easier to know how much the process took, in case you want additional logs of the timer.

Outer2g

Will you be willing to accept a PR adding tag support? http://docs.datadoghq.com/guides/dogstatsd/ ``` metric.name:value|type|@sample_rate|#tag1:value,tag2 ```

catamphetamine

UDP package length is 1472 bytes. This happened when using `send` API to send a batch of large metrics. Is it ok if I submit a PR?

vinhlh

Previous phrasing was misleading. I noticed lynx was giving uncaught exceptions and realized emitting errors was the default error handling.

elhoyos

Fix incorrect loop through array

2

I think we should be looping through the elements of the array, not its properties.

alexcole

Update README info about timers

1

I saw the comment about timers depending on `Date.getTime()` and was planning a PR to use `process.hrtime`, but found it was already done :) This PR updates the README to...

paulwib

"Nothing to send" spam

5

Why is it considered an error if there's no stats to send? If you're sending metrics with a sample rate lower than 1.0, you end up with the message "Nothing...

samcday

Doesn't work with cluster

3

Hello guys, Very strange situation: I sent timers from [workers](http://nodejs.org/docs/v0.11.11/api/cluster.html) but after few timers (statsd success receive) master is down with error: ``` assert.js:98 throw new assert.AssertionError({ ^ AssertionError: false...

shumkov