bernd / statsd-influxdb-backend

A naive InfluxDB backend for StatsD
ISC License
172 stars 74 forks source link

why not support for sets? #4

Closed Tassemble closed 9 years ago

Tassemble commented 10 years ago

why Flush Strategy could not support for statsd sets ?

Tassemble commented 10 years ago

just only get the set[key] size ? @bernd

v9n commented 9 years ago

Hi, do u have a solution for this? I'm going to add support for Sets.

v9n commented 9 years ago

@Tassemble @bernd I need this urgently and fork it https://github.com/kureikain/statsd-influxdb-backend? Will you accept a PR?

bernd commented 9 years ago

@bernd Sure, I'll accept PRs. :+1:

v9n commented 9 years ago

@bernd I submit a PR, let me know if it's something you will consider then I will do some cleanup.

bernd commented 9 years ago

I merged the PR from @kureikain for SET support today. Will cut a release soon. Thank you!