Group By implementation not thread safe

[dotnet]

Group By implementation not thread safe

[mattpodwysocki]

davedev wrote Jul 3, 2013 at 7:50 PM

Hi,

That is the correct behavior.

One of Rx's contracts is that notifications must be pushed serially; i.e., you must not invoke OnNext concurrently within a single observable. There are probably many Rx operators that may cause threading bugs if you do not satisfy this contract.

See §4.2 in the Rx Design Guidelines document.
◾Dave

[mattpodwysocki]

Bnaya wrote Aug 10, 2013 at 3:15 PM

Isn't it time to re-consider this limitation?
it is true that this is the current contract, but many operatoir can be thread-safe with
a little effort.
I believe that it should change, maybe by adding a set of thread-safe operators.
without doing it there is a need for synchronization and we may not be able to get full
utilization of a multi core environment (see Amdahl's law).

[mattpodwysocki]

davedev wrote Aug 10, 2013 at 6:54 PM

I reject the premise. A thread-safe operator is thread safe because it ensures synchronization, so whether Rx does it or you do it yourself (e.g., via the Synchronize operator) it's going to end up being the same thing anyway. A set of thread-safe operators would essentially be, perhaps, identical to the set of existing operators with the addition of the Synchronize operator.