-
Notifications
You must be signed in to change notification settings - Fork 476
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
Sending multiple queries in a single API call broken in 2.4 #214
Comments
Can you write a test which shows this behavior ? This is actually not part of our unit-test otherwise it would have been seen. @bentatham any ideas ? |
Any updates on this, as this scares me a bit. |
Nothing comes to mind. I don't seen anything explicit in the code about separating based on |
Any ETA on fixing this? One of our use cases relies on it quite heavily. Thanks in advance |
okhttp does not encode curl "http://127.0.0.1:8086/query?db=testdb&q=select%20*%20from%20notexist%3Bselect%20*%20from%20notexist" the result should be {"results":[{},{}]} but with {"results":[{}]} |
Is this on the request or response side ? |
the request side.
there is a {"results":[{}]} sorry for my poor English. @Test
public void testMultipleQuery() {
String dbName = "unittest_" + System.currentTimeMillis();
this.influxDB.createDatabase(dbName);
QueryResult result = this.influxDB.query(new Query("SELECT * FROM not_exists;SELECT * FROM not_exists", dbName));
Assert.assertEquals(2, result.getResults().size());
this.influxDB.deleteDatabase(dbName);
} This unit test should fail! |
The issue is that okhttp3 for browsers' consistency simply doesn't encode Then InfluxDB backend in addition to the encoded |
Sending multiple queries to InfluxDB in a single API call is broken in 2.4 (delimiting each query using a semicolon, see example). It only returns the first result, where in 2.3 it worked flawlessly.
Guys, what has changed?
The text was updated successfully, but these errors were encountered: