influxdata / influxdata/influxdb-java
InfluxDBException: 'error parsing query' - Error is not propagated to Consumer callback
- Vorherrschende Sprache
- Java
- Sterne
- 1.2k
- Forks
- 469
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Hello,
please take a look at [InfluxDBImpl.java#L548](https://github.com/influxdata/influxdb-java/blob/master/src/main/java/org/influxdb/impl/InfluxDBImpl.java)
If you throw an InfluxDBException an this position, it is not possible to catch it on the caller side:
```
public void loadDataAsync(int chunk, Consumer consumer) {
// Bad syntax test
db.query(new Query("SELECT * FROM sieries LIMIT -100", databaseName), chunk, consumer);
}
```
Stacktrace:
> org.influxdb.InfluxDBException: {"error":"error parsing query: found -, expected integer at line 1, char 63"}
>
> at org.influxdb.impl.InfluxDBImpl$1.onResponse(InfluxDBImpl.java:420)
> at retrofit2.OkHttpCall$1.callSuccess(OkHttpCall.java:138)
> at retrofit2.OkHttpCall$1.onResponse(OkHttpCall.java:117)
> at okhttp3.RealCall$AsyncCall.execute(RealCall.java:141)
> at okhttp3.internal.NamedRunnable.run(NamedRunnable.java:32)
> at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
> at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
> at java.lang.Thread.run(Thread.java:748)
This causes my REST-Endpoint (Spring MVC) to hang.

Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Rechercherichtung
Beginne in src/main/java/org/influxdb/impl/InfluxDBImpl.java rund um die referenzierte Zeile und verfolge die Verarbeitung der asynchronen Query-Antwort, die im Stack Trace angezeigt wird. Reproduziere den Fall einer ungültigen Query und überprüfe anschließend, dass der Consumer-Callback den Fehler empfängt und der aufrufende REST-Endpunkt nicht mehr hängen bleibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java
- Bereich
- api
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100