Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to query against columns with dots in their names #240

Merged
merged 1 commit into from
Feb 10, 2014

Conversation

jvshahid
Copy link
Contributor

There is some problem querying columns that have dots (and I guess other 'special' symbols in their names).

I've created a single point with two columns - a.1 & a.2 in a series called foo.

When I request all the data (*), I do get proper result:

curl 'http://localhost:8086/db/metrics/series?u=root&p=root&q=select+*+from+foo'

[{"name":"foo","columns":["time","sequence_number","a.1","a.2"],"points":[[1391713357450,487020001,1,2]]}]

When I request the first column (a.1) everything still looks good:

curl 'http://localhost:8086/db/metrics/series?u=root&p=root&q=select+a.1+from+foo'

[{"name":"foo","columns":["time","sequence_number","a.1"],"points":[[1391713357450,487020001,1]]}]

When I request the second column (a.2) though, unexpected stuff happens (data for a.1 is returned):

curl 'http://localhost:8086/db/metrics/series?u=root&p=root&q=select+a.2+from+foo'

[{"name":"foo","columns":["time","sequence_number","a.1"],"points":[[1391713357450,487020001,1]]}]

@FGRibreau
Copy link

👍 just had the same issue, we are storing columns with dot inside their names (since influxdb does not support nested objects) we can request them all but not a subset of them.

Don't always assume that column names with dots have the format
`table_name.column_name`. Instead, we check whether `table_name` exists
and if it doesn't we treat the full name as a column name.
pauldix added a commit that referenced this pull request Feb 10, 2014
Unable to query against columns with dots in their names
@pauldix pauldix merged commit 2c8ee2e into master Feb 10, 2014
@FGRibreau
Copy link

Thanks for the merge into master @pauldix !

@jvshahid jvshahid deleted the fix-240-column-names-with-dots branch March 3, 2014 23:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants