This repository was archived by the owner on Oct 29, 2024. It is now read-only.
This repository was archived by the owner on Oct 29, 2024. It is now read-only.
retry on HTTPError exceptions need raise_for_status()? #832
Open
Description
We occasionally receive a http status 500 code (time out) from our influxdb instance, probably because of temporary peak load. Looking at the retry code in InfluxDBClient class I can see that you try to catch HTTPErrors (line 345):
influxdb-python/influxdb/client.py
Lines 326 to 353 in fc0235e
According to the requests library documentation a raise_for_status() is needed for the exception to be caught:
https://requests.readthedocs.io/en/master/user/quickstart/#errors-and-exceptions
Or am I missing something?