-
Notifications
You must be signed in to change notification settings - Fork 72
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
InfluxQL with Influx 1.8 #195
Comments
Thank you for reporting this issue and describing your workaround. InfluxDB1.8 server now does not support influxql queries in 2.0 compatibility endpoints that are used by this client, the code accepts only flux queries. The 2.0 documentation mentions only flux queries, even though the 2.0 API offers the influxql query option, the support of influxql in 2.0 is not documented, therefore it is not something to rely upon. Your query would fail also in 2.0:
This client shall be used to post I would better remove the |
Thanks for the clarification
Yes I agree The reason for trying this client is the statement:
Since we already have an existing nodejs application using the current node-influx client I was thinking it would be nice to prepare our self for the new 2.0 series. Using the compatibility mode by using I will put our migration on hold for now, since we are running Influx on Windows environment and at the moment there are no binaries for 2.0. So no (easy) adoption for the 2.0 with Influx. Issue can be closed from my point of view. |
Expected Behavior
If I define type
influxql
I should be able to do a simple queryCurrent Behavior
Response with a bad request
const queryApi = this.influxClient.getQueryApi('').with({ type: 'influxql' });
Error log:
Possible Solution
Accept
influxql
since I want to do an easy query:SHOW DATABASES
since with 1.8 thebuckets()
doesn't work.I can get a proper response by:
Steps/Code to Reproduce the Problem
Specifications
The text was updated successfully, but these errors were encountered: