Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Always use savepoints when nesting transactions #1135

Merged
merged 3 commits into from
Oct 2, 2023

Conversation

mvorisek
Copy link
Member

@mvorisek mvorisek commented Oct 2, 2023

the only supported mode in upcoming DBAL 4.x

no BC break when the transaction is not rolled back or rolled completely

@mvorisek mvorisek force-pushed the transaction_savepoints branch 3 times, most recently from a0812b6 to abcea03 Compare October 2, 2023 12:51
@mvorisek mvorisek marked this pull request as ready for review October 2, 2023 12:53
@mvorisek mvorisek changed the title Allow start transaction during result iteration for MSSQL Always use savepoints when nesting transactions Oct 2, 2023
@mvorisek mvorisek marked this pull request as draft October 2, 2023 12:59
@mvorisek mvorisek force-pushed the transaction_savepoints branch from da1a958 to 5ec6236 Compare October 2, 2023 13:20
@mvorisek mvorisek marked this pull request as ready for review October 2, 2023 13:22
@mvorisek mvorisek merged commit 947664c into develop Oct 2, 2023
31 checks passed
@mvorisek mvorisek deleted the transaction_savepoints branch October 2, 2023 13:44
@mvorisek mvorisek added MAJOR and removed BC-break labels Oct 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant