I have something similar to this in my code:
Parallel.ForEach(myList, new ParallelOptions { MaxDegreeOfParallelism = 4 }, item =>
{
Process(item);
});
I ended up with a slightly different implementation. In my case, checking for the CancellationToken
within Process()
wouldn't work due to potentially long running statements in between checks. For example if my timeout was 5 seconds, and a single statement took say 100 seconds ... I wouldn't know until that statement completed and then was detected by if (token.IsCancellationRequested)
.
Here's what I ended up doing
Parallel.ForEach(myList, (item) =>
{
Task task = Task.Factory.StartNew(() =>
{
Process(item));
});
if(task.Wait(10000)) // Specify overall timeout for Process() here
Console.WriteLine("Didn't Time out. All's good!");
else
Console.WriteLine("Timed out. Leave this one and continue with the rest");
});
Then within the Process()
method, I added further checks on potentially long running statements to allow it to gracefully handle timeouts (as much as possible). So it's only in the worst case that Process() had to be prematurely stopped by Task.Wait()
above.
private void Process(MyItem item)
{
...
cmd.CommandTimeout = 5; // The total of timeouts within Process() were
// set to be less than the total Task.Wait duration.
// Unfortunately some potentially long running methods
// don't have a built in timeout.
...
}