Question

We have a Windows Service written in C#. The service spawns a thread that does this:

private void ThreadWorkerFunction() {   while(false == _stop) // stop flag set by other thread   {     try     {       openConnection();        doStuff();        closeConnection();     }     catch (Exception ex)     {       log.Error('Something went wrong.', ex);        Thread.Sleep(TimeSpan.FromMinutes(10));     }   } } 

We put the Thread.Sleep in after a couple of times when the database had gone away and we came back to 3Gb logs files full of database connection errors.

This has been running fine for months, but recently we’ve seen a few instances where the log.Error() statement logs a ‘System.InvalidOperationException: This SqlTransaction has completed; it is no longer usable’ exception and then never ever comes back. The service can be left running for days but nothing more will be logged.

Having done some reading I know that Thread.Sleep is not ideal, but why would it simply never come back?

Answer 1

Dig in and find out? Stick a debugger on that bastard!

I can see at least the following possibilities:

1. the logging system hangs;
2. the thread exited just fine but the service is still running because some other part has a logic error.

And maybe, but almost certainly not, the following:

• Sleep() hangs.

But in any case, attaching a debugger will show you whether the thread is still there and whether it really has hung.