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
feat: add nested transaction #8541
feat: add nested transaction #8541
Conversation
Thank you for your contribution, looks really good. Considering simplicity of the suggested solution, I just wanted to ask you if you have a time to investigate other drivers as well, because: cockroachdb - should be same as postgres if they support it |
sure. I'll give it a try tomorrow |
This will allow nested transaction for postgres driver using "save point" Closes: #1505
I added other drivers except for mssql. I wasn't able to implement it :( |
code refactoring;
thank you for contribution! |
I try this example and it still doesn't work for me :-( |
Is mssql still not capable of using this feature? |
Description of change
Add nested transaction support.
Implemented following drivers:
Following drivers also support nested transaction. but I'm not familiar with them:
May be someone else can implement it in another PR
Fixes: #1505
Pull-Request Checklist
master
branchnpm run lint
passes with this changenpm run test
passes with this changeFixes #1505