search

kdcllc / CometD.NetCore

CometD for use with Salesforce Platform Events
MIT License
28 stars 16 forks source link

Should StateUpdateInProgressMutex really be a static Mutex? #14

Closed AlexVallat closed 3 years ago

AlexVallat commented 3 years ago

It seems to me that this exclusion has no reason to synchronise across multiple instances of BayeuxClient, surely each should be independent? The _stateUpdateInProgress variable it protects is not static.

AlexVallat commented 3 years ago

(I only mention this because when debugging I sometimes get into a state where it throws AbandonedMutexException and never recovers from it, even when disposing and creating a new BayeuxClient)

kdcllc commented 3 years ago

@AlexVallat thanks for your contribution!