You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Dec 20, 2022. It is now read-only.
Current implementation of ParallelForwardingAppender is wrong, since it block a thread form the default ThreadPool. The C# Tasks and Threads should not be mixed together. {{BlockingCollection}} is based on a thread synchronization (lock), so do not use it for the Task running on a ThreadPool.
The Task based producer - consumer is a bit more complicated, e.g. see Stephen Clearys's or .NET doc). Anyway, do not use tasks, since the ForwardLoggingEvent is not Task-based. (E.g. some appender sending logs by network may have long log times.) Just start a new plain thread.
Current implementation of
ParallelForwardingAppender
is wrong, since it block a thread form the default ThreadPool. The C# Tasks and Threads should not be mixed together. {{BlockingCollection}} is based on a thread synchronization (lock), so do not use it for the Task running on a ThreadPool.The Task based producer - consumer is a bit more complicated, e.g. see Stephen Clearys's or .NET doc). Anyway, do not use tasks, since the
ForwardLoggingEvent
is not Task-based. (E.g. some appender sending logs by network may have long log times.) Just start a new plain thread.See also #37
The text was updated successfully, but these errors were encountered: