search

vimeo / graphite-influxdb

An influxdb backend for Graphite-web and graphite-api
Apache License 2.0
198 stars 39 forks source link

Add replace for graphite's {a,b} regular expressions. #27

Closed vladimir-smirnov-sociomantic closed 10 years ago

vladimir-smirnov-sociomantic commented 10 years ago

Should fix #26

Dieterbe commented 10 years ago

I don't think you can just replace every comma with a pipe, that would break functions with multiple arguments, no? i think you need some more advanced regex here.

vladimir-smirnov-sociomantic commented 10 years ago

I think no, because splitting function arguments done by graphite-web/graphite-api/whatever, and graphite-influxdb receives only list of queries.

Dieterbe commented 10 years ago

thanks! note: i personally have not tested it (yet), but it looks right, so i don't want to hold this up.