Replace queries with single service class#354
Replace queries with single service class#354stefanolsen wants to merge 3 commits intovaldisiljuconoks:v9from
Conversation
…rewrite-queries # Conflicts: # common/src/DbLocalizationProvider/LocalizationProvider.cs # common/src/DbLocalizationProvider/ReflectionConverter.cs
|
@valdisiljuconoks This is WIP to show the idea early on. Is this refactoring fine by you? Note: test cases are not yet updated. |
|
hi, idea with composition of queries and commands was - to be able to override single command and have different behavior. we need to measure what is perf. implications for cqrs vs plain repos. |
|
Isn't it okay to override the whole repository class? That way we are using the default service container. With the CQRS model, it seems to require a custom resolver. |
|
@valdisiljuconoks Is it not open enough for overrides when it is in a service backed by a public interface? |
This PR seeks to replace the number of queries and query handlers with a single service class.
This way instantiating the service graph is simpler, methods can be overridden and we end up using the default dependency injection framework.
This is what is referred to in #345.