influxdata / influxdata/influxdb-java

It should be possible to not log errors in BatchProcessor

Aperta
#401 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub
kind/enhancement
Lingua principale
Java
Stelle
1.2k
Fork
469
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

The current exception handler implementation doesn't make it possible to completely eliminate logging of errors. You can add an additional error handler, but `LOG.log(Level.SEVERE, "Batch could not be sent. Data will be lost", t);` will still be run no matter what. We would like to only log errors periodically so we don't completely spam our logs from all servers if InfluxDB goes down temporarily.

Could the default logging be implemented as a default exception handler that can be replaced or enhanced? I could turn off the logging, but that would mean disabling all logging from this class.

Another option would be to allow for turning down the log level of these errors, maybe to DEBUG or some other setting less than error.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Inizia individuando BatchProcessor e il gestore delle eccezioni che contiene la chiamata incondizionata LOG.log(Level.SEVERE, "Batch could not be sent. Data will be lost", t). Esamina come sono configurati i gestori degli errori aggiuntivi e il logging della classe, quindi definisci il completamento in modo da consentire che questo logging degli errori predefinito venga sostituito, migliorato o assegnato a un livello inferiore senza disabilitare il logging non correlato.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
java
Ambito
api
Tipo di issue
Funzionalità
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.