Question

Just read this interesting article by Omar on his blog Linq to SQL solve Transaction deadlock and Query timeout problem using uncommitted reads and at the end Javed Hasan started arguing with him about his solution to the nolock situation on a high volume site.

Here, the problem trying to solve is, from the sql sense we need to use Select statements with NOLOCK or use SET TRANSACTION LEVEL READ UNCOMMITTED, otherwise at high volume rows in DB will be locked and cause errors. The technology Omar used is Linq2Sql, so the question is how do we get this achieved in your C# data access code so the above does not happen?

Basically in the post, Omar comes to his solution by working and testing on real world site and with tools like SqlProfiler, whereas Javed Hasan comes to his solution with MSDN documents and Scott Hanselman’s blog post etc.

Omar suggests using the following

using (var db = new DropthingsDataContext2()) {   db.Connection.Open();   db.ExecuteCommand('SET TRANSACTION ISOLATION LEVEL READ UNCOMMITTED;');    var user = db.aspnet_Users.First();   var pages = user.Pages.ToList(); } 

whereas Javed Hasan suggests

using (new TransactionScope(TransactionScopeOption.Required,    new TransactionOptions { IsolationLevel = IsolationLevel.ReadUncommitted })) {  //Your db Operation } 

I’m very interested to know what you guys out there do on this particular issue on a high volume site like the StatckOverflow, or what did Jeff and their guys do in this regard?

Edit: After reading the first post, I want to point out a few things in Omar’s post.

1. he did run into connection problem with his approach but he solved it, see his post.
2. more importantly he mentioned he tried the using ADO.NET Transaction way and even tried what Scott Hanselman wrote on his blog, but it is not working for high volume site, it degrades the performance quite a bit. Omar said this ‘System.Transactions has significant overhead. I have never been able to use it on a high volume website without making CPU go 100% and Req/sec come down to 1/10th. It’s made for enterprise applications, not for high volume websites.’

Answer 1

First of all please avoid uncommitted reads, they can cause lots of issues. A much better approach is just to set the database to snapshot isolation. This is what Jeff did.

Jeff basically said: ‘bla bla bla, be real, bla bla bla, database theoreticians, bla bla bla, READ UNCOMMITTED can be useful for REAL production apps that don’t need data consistency.’ Jeff is not a DBA, fortunately there are many DBAs out here on SO.

The problem with Omar’s approach is that it can leak connections with ‘read uncommitted’ isolation level in to your connections pool which could wreak havoc in your website. Meaning random statement may be executed in read uncommitted.

Javed approach would be much better because on dispose MS have the chance to clean stuff up on the connection.

EDIT If you are having performance issues with Javed’s approach you could look at rolling your own transaction manager.

Some things you probably want to do:

• Hold a stack of current transactions
• Confirm you are on the creator thread when a transaction is committed
• Reset the transaction isolation to its previous state on dispose
• Rollback on dispose if the transaction was not committed.
• Support nested rollbacks.