-
Notifications
You must be signed in to change notification settings - Fork 7
The library is not thread safe when sending metric #10
Open
Description
We found out an exception like below:
System.InvalidOperationException: This BackgroundWorker is currently busy and cannot run multiple tasks concurrently.
at System.ComponentModel.BackgroundWorker.RunWorkerAsync(Object argument)
at StatsN.BaseCommunicationProvider.SendMetric(String metric)
at StatsN.Statsd.LogMetricAsync(String metricName, String value, String metricType, String postfix)
Looks like in BaseCommunicationProvider.SendMetric,
if (!worker.IsBusy) worker.RunWorkerAsync();
It does check if the worker is busy, but looks like this may not be thread safe.
Metadata
Metadata
Assignees
Labels
No labels
Type
Fields
Give feedbackNo fields configured for issues without a type.