AsyncPoco icon indicating copy to clipboard operation
AsyncPoco copied to clipboard
verified publisher icon tmenier

Added IsolationLevel optional parameter to GetTransactionAsync and Begin...

Open pavick opened this issue 10 years ago • 3 comments

...TransactionAsync

The default isolation level is read committed.

pavick avatar Mar 25 '15 18:03 pavick

Sorry about the delay. I've not seen it documented that ReadCommitted is the default isolation level for any db other that SQL Server, so I'm not sure I agree with the implementation. I'm on board with the idea but I think it needs to defer to the provider for the default rather than hard-coding to ReadCommitted.

tmenier avatar May 09 '15 16:05 tmenier

I understand. In that case the easiest thing to do would be to overload the Transaction methods with methods that have a required IsolationLevel parameter.

pavick avatar May 09 '15 16:05 pavick

Sorry for dropping the ball on this. If it's still something you're interested in I'm willing to accept this and make a few tweaks so as not to assume ReadCommitted is the default. Let me know.

tmenier avatar Aug 12 '16 19:08 tmenier