Question : logs or throwing exceptions ?
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 25/100
Rechercherichtung
Das Issue verweist auf die Java-Methode ready(), ihren Aufruf execute(call) und den InfluxException-Handler. Beginne damit, diese Einstiegspunkte nachzuverfolgen und das Verhalten der umliegenden Protokollierung zu vergleichen; als erledigt gilt die Aufgabe, wenn das Projekt ein festgelegtes Verhalten für die Fehlerberichterstattung hat, das die Details von Verbindungsfehlern bewahrt und gleichzeitig ermöglicht, Protokolle zu unterdrücken.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Question:
Is there a reason we are logging here and other places, instead of throwing an exception ?
I'd be intersted in suppressing logs from InfluxDb but i still want to be able to know what kind of connection error I get.
Is it too expensive to throw exceptions?
Best regards
@Nullable
public Ready ready() {
Call<Ready> call = this.readyService.getReady((String)null);
try {
return (Ready)this.execute(call);
} catch (InfluxException e) {
LOG.log(Level.WARNING, "The exception occurs during check instance readiness", e);
return null;
}
}
- Vorherrschende Sprache
- Java
- Sterne
- 477
- Forks
- 129
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Kein Beitragsleitfaden
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus influxdata/influxdb-client-java
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 52/100
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
influxdata/influxdb-client-java#849 · 4 Kommentare ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100
influxdata/influxdb-client-java#830 · 1 Kommentar ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 38/100
-
InfluxQLQueryApiImpl cannot cast JsonNullEvtl. wieder frei @NguyenHoangSon96 hat das vor 605 Tagen übernommen, und es ist kein Pull Request offen. Offen
influxdata/influxdb-client-java#808 · 3 Kommentare · 1 zugewiesene Person ·
Alle Issues in influxdata/influxdb-client-java
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
beehive-lab/TornadoVM#1151 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 77/100
FasterXML/jackson-dataformats-binary#823 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag