kefirfromperm/grails-asynchronous-mail

ExpiredMessagesCollectorJob should not contain any SQL

Closed this issue · 1 comments

stokito commented

ExpiredMessagesCollectorJob updates AsynchronousMailMessage table.
It looks for me not a good idea. There is a convention that all changes to DB should be done by service.
Could you extract this code to method in AsynchronousMailPersistenceService class?

stokito commented

I found your commit 992c06a that fix problem.
Thanks!