docbee-fschrader commented on issue #15276: URL: https://github.com/apache/grails-core/issues/15276#issuecomment-3660832625
Thanks for searching the issue that caused the change, I can see why a change was necessary. There are only a few cases in the application I'm working on where invalid data is saved. I'd be fine discarding the domain in that specific case. However we have a lot of cases where we skip validation because some of our validators are expensive and we know the object we're saving is valid. It would be very inconvenient (and bad for our code quality) to add a discard in all those places. That's why I was asking, whether it is really necessary to validate all domains (including the ones saved with validate:false), when the transaction is flushed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
