forked from datastax/python-driver
-
Notifications
You must be signed in to change notification settings - Fork 49
Open
Labels
enhancementNew feature or requestNew feature or requestupstream-issueIssue is not specific to our fork and Scylla and should be also fixed in the upstreamIssue is not specific to our fork and Scylla and should be also fixed in the upstream
Description
In the driver's code there are places where we schedule an asynchronous task but ignore its result.
This can lead to silently ignoring exceptions thrown by the async task (e.g. c7e6ebb). Such behaviour is dangerous. It could lead to hours wasted on the debugging by users of the driver. It could even be the cause of some internal driver bugs.
There are multiple ways to approach the issue. I believe we should find the solution which not only gets rid of current occurrences of the problem, but also prevents the developers from introducing similar bugs in the future. One idea might be to introduce some wrapper over (Green)ThreadPoolExecutor
which would log some message in case the async task threw an exception.
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requestupstream-issueIssue is not specific to our fork and Scylla and should be also fixed in the upstreamIssue is not specific to our fork and Scylla and should be also fixed in the upstream