influxdata / influxdata/influxdb-java

InfluxDBException: 'error parsing query' - Error is not propagated to Consumer callback

Offen
#498 0 Kommentare 2 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
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.
![image](https://user-images.githubusercontent.com/10603718/44657325-bba7f700-a9fc-11e8-8bec-1d496b6ce027.png)

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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.