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

fix(core/query): remove influxql support #195 #196

Merged
merged 2 commits into from
Jun 4, 2020

Conversation

sranka
Copy link
Contributor

@sranka sranka commented Jun 4, 2020

Fixes #195

This PR removes support for influxql queries in the query API, there are multiple reasons for doing this:

  • the state of support of influxql queries in InfluxDB 2.0 is unknown and not documented
  • the server returns json data to influxql query, such data is not parseable by this client that expect CSV data as the only option explained by the docs
  • even 1.x server does not support influxql in 2.0 compatibility API

Checklist

  • CHANGELOG.md updated
  • A test has been added if appropriate
  • yarn test completes successfully
  • Commit messages are in semantic format
  • Sign CLA (if not already signed)

@sranka sranka requested a review from bednar June 4, 2020 04:47
@sranka sranka merged commit 53aa235 into master Jun 4, 2020
@sranka sranka deleted the 195/remove_influxql_query branch June 4, 2020 11:20
@bednar bednar added this to the 1.4.0 milestone Jun 4, 2020
@sezanzeb
Copy link

sezanzeb commented May 22, 2023

This might be outdated, and I would like influxql support to be added back in, because it responds data in "wide" format, as opposed to the "narrow" format of flux

the state of support of influxql queries in InfluxDB 2.0 is unknown and not documented

InfluxQL is supported in 2.x (see below) and even 3.x: https://www.influxdata.com/blog/introducing-influxdb-3-0/

the server returns json data to influxql query, such data is not parseable by this client that expect CSV data as the only option explained by the docs

By setting the Accept header data is returned in csv

Example:

curl --request POST \
        "http://localhost:8086/query"  \
        --header 'Authorization: Token testtoken' \
        --header 'Accept: application/csv' \
        --data "q=SELECT * FROM testproject..bar WHERE device = 'foo'"

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.

InfluxQL with Influx 1.8
3 participants