SqlDataReader and concurrency/deallocation issues

Question

Running the below code is giving me an error:

.NET Framework execution was aborted by escalation policy because of out of memory. System.InvalidOperationException: There is already an open DataReader associated with this Command which must be closed first.

I have the following code all over the place to run my sql:

sql.Append(string.Format("SELECT TableId FROM ps_SavedTables WHERE guid = '{0}'", guid));

    using (IDataReader reader = SqlHelper.GetDataReader(sql.ToString())) {
        if (reader.Read()) {
            result = reader.IsDBNull(0) ? string.Empty : reader[0].ToString();
        }
        //CDW added to close SqlDataReader
        reader.Close();
    }

The GetDataReader is defined by this:

public static SqlDataReader GetDataReader(string sql, string connectionString) {
    lock (_lock) {
        SqlConnection connection = null;
        try {
            connection = GetConnection(connectionString);
            //connection.Open();
            using (SqlCommand cmd = new SqlCommand(sql, connection)) {
                WriteDebugInfo("GetDataReader", sql);
                return cmd.ExecuteReader(CommandBehavior.CloseConnection);
            }
        }
        catch (Exception e) {
            if (connection != null)
                connection.Dispose();
            throw new DataException(sql, connectionString, e);
        }

    }
}

Can you give us a bit more detail about the exception? Where it's thrown? Stack trace etc? — Dave Lawrence
– Dave Lawrence, Commented May 23, 2012 at 8:23

Dave Lawrence · Accepted Answer · 2012-05-23 08:29:40Z

1

The lock needs to be at the reader level... Multiple threads are opening readers

answered May 23, 2012 at 8:29

Dave Lawrence

3,9592 gold badges23 silver badges36 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Dave Lawrence Over a year ago

@Totero I'm glad someone had the manners to comment. Thanks. Poor stuff from the op. Spam merchant.

Dave Lawrence Over a year ago

Oh you like what he's selling! It brings the boys to the yard apparently

Collectives™ on Stack Overflow

SqlDataReader and concurrency/deallocation issues

1 Answer 1

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related