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.

Sign up for GitHub

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

Jump to bottom

Run all hooks within transactions #27

Open

overlookmotel opened this issue Sep 10, 2015 · 1 comment

Labels

Owner

overlookmotel commented Sep 10, 2015

All hooks should run within a transaction, and create a new transaction if none exists already.

overlookmotel added the enhancement label

Owner Author

overlookmotel commented Jul 11, 2016

All hook code should run within a transaction along with the main sequelize code (if this is possible with current Sequelize hooks API).

Problem is that with the hooks, an action runs multiple SQL statements, but consistency cannot be guaranteed.

overlookmotel mentioned this issue

Run all hooks within transactions #57

Closed

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