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

Syntax for setting transaction isolation level #1078

Closed
vieks opened this issue Nov 26, 2015 · 6 comments
Closed

Syntax for setting transaction isolation level #1078

vieks opened this issue Nov 26, 2015 · 6 comments

Comments

@vieks
Copy link

vieks commented Nov 26, 2015

I don't see it in the doc how to set isolation level for transactions ?

Thx.

@elhigu
Copy link
Member

elhigu commented Nov 30, 2015

As far as I know there is no way to do it without knex.raw().

@vieks
Copy link
Author

vieks commented Nov 30, 2015

Why not implement it as a parameter for the transaction method ? It will be a quick and more elegant way than doing a raw call.

Like Sequalize do: http://docs.sequelizejs.com/en/latest/docs/transactions/#isolation-levels

@elhigu
Copy link
Member

elhigu commented Nov 30, 2015

Something like that sounds reasonable, exact syntax needs to be discussed though. When it is clear what is wanted, someone might implement it too :) Do you have some clear vision how it would exactly work with knex syntax to get discussion started?

@elhigu elhigu changed the title Transaction Syntax for setting transaction isolation level Nov 30, 2015
@carera
Copy link

carera commented Jan 6, 2016

+1 for this
For now, can someone tell me how would i approach this with knex.raw(), please?

@jmurzy
Copy link

jmurzy commented Dec 13, 2016

This is a dupe of #581.

@carera You must have figured it out by now but the current "workaround" for Posgres is in #581. A bit more hackier but I believe you could do the same for MySQL as well.

@elhigu
Copy link
Member

elhigu commented Dec 21, 2016

@jmurzy thanks! Closing this one...

@elhigu elhigu closed this as completed Dec 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants