You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Not all services need db support Service class should not have a TransactionalSupport as dependency. There could a separate DbService or TransactionalSupport can be done as a decorator/mixin,
The text was updated successfully, but these errors were encountered:
I removed the SqlSessionMixin because of redundancy with
TransactionalSupport class, which Martin Babka added as dependency in
Service. My original intention was not add as a direct ancestor to
Serivce and have more like a mixin.
I still think that having a plain Service (without dependencies) would
be better, not all services needs DB access.
See #15
Not all services need db support Service class should not have a TransactionalSupport as dependency. There could a separate DbService or TransactionalSupport can be done as a decorator/mixin,
The text was updated successfully, but these errors were encountered: