LucidSage/MassTransit — comment on commit bced6b707beb1644b07b51e62b65579635c95c07
File:  line 0
By LucidSage:

I have been deliberately going out of my way to keep my changes completely separate. I'm new to this code so I am just being cautious. I also did not want to make any assumptions about where it should be put until it was a little more mature.

It does duplicate the distributor code, because it needs to do some of the same things and so that's what I started with, but it is really a replacement for the MessageRouter. This deeper integration is something that the current Distributor does not do, so I am not sure if extending the Distributor is a good fit. Although your idea may address this, I'd be worried about breaking people that are depending on it working as is.

Maybe calling my changes a Distributor isn't even right. Maybe it should be GroupMessageRouter instead. It needs some cleanup too. There are some parts that I don't think are getting used like pending message counts. These changes will make it diverge from the Distributor even more as I continue to refine it.

It also needs saga support, or at least testing. I do not have a good understanding of those yet so I have been putting that off for now. Any guidance for this feature would be appreciated!

I am also curious to know if my approach is just going down the wrong path or if you see some merit to this idea?