Skip to content

OBGM-574 Disable allow_update_outside_transaction Hibernate setting#4136

Merged
kchelstowski merged 1 commit intofeature/upgrade-to-grails-3.3.10from
OBGM-574
Jul 4, 2023
Merged

OBGM-574 Disable allow_update_outside_transaction Hibernate setting#4136
kchelstowski merged 1 commit intofeature/upgrade-to-grails-3.3.10from
OBGM-574

Conversation

@kchelstowski
Copy link
Collaborator

@kchelstowski kchelstowski commented Jul 4, 2023

I know I could've deleted this setting at all, but I prefer to leave it with the false flag, because figuring out what Hibernate settings for an exact version are enabled/disabled by default, is a pain in the neck, so it will provide a better readability and understanding, what is enabled/disabled

@kchelstowski kchelstowski self-assigned this Jul 4, 2023
@kchelstowski kchelstowski requested a review from alannadolny July 4, 2023 06:21
@kchelstowski
Copy link
Collaborator Author

I'm merging this one - the faster we find bugs related to that, the better.

@kchelstowski kchelstowski merged commit 6cf89df into feature/upgrade-to-grails-3.3.10 Jul 4, 2023
@kchelstowski kchelstowski deleted the OBGM-574 branch July 4, 2023 06:29
Copy link
Member

@jmiranda jmiranda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree with @kchelstowski that we should be setting configuration explicitly (with very descriptive commit messages).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants