search

obfuscurity / tasseo

Live dashboard for Graphite
Other
1.53k stars 127 forks source link

Fix InfluxDB query to latest #73

Closed client9 closed 11 years ago

client9 commented 11 years ago

the new query syntax is apparently select * from _metricname ...

obfuscurity commented 11 years ago

Where are you getting this from? I just tested against the InfluxDB sandbox and the "old style" query still works for me. /cc @pauldix

screen shot 2013-11-14 at 9 03 49 pm

client9 commented 11 years ago

from the latest deb package for ubuntu (downloaded yesterday, which I know matches HEAD since it fixed a bug for me) and from their documentation.

I have not used the sandbox.

obfuscurity commented 11 years ago

Yeah, there's no guarantee that the sandbox is running the newest. Have you tested this change against InfluxDB master?

obfuscurity commented 11 years ago

My main concern is that this query will return multiple metrics.

client9 commented 11 years ago

worked for me, on master, although I was using a dataset w/ columns of “time”, “value” only (coming from Diamond/collectd type of system)

2013/11/15 11:10、Jason Dixon notifications@github.com のメール：

My main concern is that this query will return multiple metrics.

— Reply to this email directly or view it on GitHub.

obfuscurity commented 11 years ago

Curious, which docs do you see this in? The online docs still specify "select metric from" when pulling a single series.

obfuscurity commented 11 years ago

Installed latest on a test system and confirmed that the original query still still works. However I did uncover a bug in that I'd hardcode my test series "data" in the query. I'll need to add support for that in the configuration syntax.

client9 commented 11 years ago

Ahh that's what it must have been... I'm New to your code and influxdb, I just assumed the query format was wrong. Thanks for your time

Sent from a magical device

On 2013/11/15, at 12:52, Jason Dixon notifications@github.com wrote:

Installed latest on a test system and confirmed that the original query still still works. However I did uncover a bug in that I'd hardcode my test series "data" in the query. I'll need to add support for that in the configuration syntax.

\ Reply to this email directly or view it on GitHub.

obfuscurity commented 11 years ago

Sure thing. Sorry for the :bug:.