search

uber-archive / node-statsd-client

Node.js client for statsd
ISC License
28 stars 9 forks source link

[DNS recovery]: add a packet queue for coalescion. #10

Closed Raynos closed 9 years ago

Raynos commented 9 years ago

This adds a packet queue to the UDP socket. This means we group packets together.

The defaults are flushing every second and flushing when we reach 1440 bytes of data.

cc @mranney @sh1mmer @kriskowal @Matt-Esch

sh1mmer commented 9 years ago

Can we fork packet queue into a separate small module and then include that instead of forking into the middle of this module?

Raynos commented 9 years ago

@sh1mmer that's a good idea.

For the sake of getting this over the finish line and in dispatch I plan to keep the packet-queue in here.

I will probably break both the packet-queue and dns-resolver modules out into seperate packages once this is deployed on monday.

jcorbin commented 9 years ago

lgtm

Raynos commented 9 years ago

@sh1mmer fixed the name.