pjvds / ncqrs

Ncqrs Framework - The CQRS Framework for .NET
Other
541 stars 162 forks source link

Rename Denormalizer component to EventHandler #80

Closed mnero closed 12 years ago

mnero commented 12 years ago

Hello,

First, I want to say that you guys have done a great job with the framework; you've saved me from writing my own! I've been reading through the documentation, source code and tutorials and I have just one comment.

I was wondering if it would be possible to rename the denormalization component of the framework to something a bit more generic. In other words, rename IDenormalizer to IEventHandler. The name IDenormalizer implies that the only kind of response to an event in your domain is to denormalize your domain to your read store, which is somewhat limiting.

In practice, I believe one could respond to an event in any number of ways. For example, one event handler could end a client's long-polling operation. So, one would have an event handler that could update your read store, and another event handler that will handle the client waiting for a response from the server.

I can do this with your framework, but it seems unnatural to handle a client's long-polling request in an IDenormalizer.

Please let me know if I've missed something.