再次遇到dotnet的第三方組件問題,就是hangfire的CoreBackgroundJobPerformer會導致死鎖,它是作為hagnfire服務端的job執行器的,它非常的關鍵,是job能夠運行的關鍵,這些庫可能讀是從很早的dotnetfremework時代移植過來的(我猜測的),同樣的存在同步調用異步代碼的問題,會導致死鎖。
它有問題的代碼如下:
namespace Hangfire.Server
{
internal sealed class CoreBackgroundJobPerformer : IBackgroundJobPerformer
{
private object InvokeMethod(PerformContext context, object instance, object[] arguments)
if (context.BackgroundJob.Job == null) return null;
try
{
var methodInfo = context.BackgroundJob.Job.Method;
var method = new BackgroundJobMethod(methodInfo, instance, arguments);
var returnType = methodInfo.ReturnType;
if (returnType.IsTaskLike(out var getTaskFunc))
{
if (_taskScheduler != null)
{
return InvokeOnTaskScheduler(context, method, getTaskFunc);
}
return InvokeOnTaskPump(context, method, getTaskFunc);
}
return InvokeSynchronously(method);
}
catch (ArgumentException ex)
{
HandleJobPerformanceException(ex, context.CancellationToken, context.BackgroundJob);
throw;
}
catch (AggregateException ex)
{
HandleJobPerformanceException(ex.InnerException, context.CancellationToken, context.BackgroundJob);
throw;
}
catch (TargetInvocationException ex)
{
HandleJobPerformanceException(ex.InnerException, context.CancellationToken, context.BackgroundJob);
throw;
}
catch (Exception ex) when (ex.IsCatchableExceptionType())
{
HandleJobPerformanceException(ex, context.CancellationToken, context.BackgroundJob);
throw;
}
}
private object InvokeOnTaskScheduler(PerformContext context, BackgroundJobMethod method, Func<object, Task> getTaskFunc)
{
var scheduledTask = Task.Factory.StartNew(
InvokeOnTaskSchedulerInternal,
method,
CancellationToken.None,
TaskCreationOptions.None,
_taskScheduler);
var result = scheduledTask.GetAwaiter().GetResult();//同步執行異步
if (result == null) return null;
return getTaskFunc(result).GetTaskLikeResult(result, method.ReturnType);
}
private static object InvokeOnTaskSchedulerInternal(object state)
{
// ExecutionContext is captured automatically when calling the Task.Factory.StartNew
// method, so we don't need to capture it manually. Please see the comment for
// synchronous method execution below for details.
return ((BackgroundJobMethod)state).Invoke();
}
private static object InvokeOnTaskPump(PerformContext context, BackgroundJobMethod method, Func<object, Task> getTaskFunc)
{
// Using SynchronizationContext here is the best default option, where workers
// are still running on synchronous dispatchers, and where a single job performer
// may be used by multiple workers. We can't create a separate TaskScheduler
// instance of every background job invocation, because TaskScheduler.Id may
// overflow relatively fast, and can't use single scheduler for multiple performers
// for better isolation in the default case – non-default external scheduler should
// be used. It's also great to preserve backward compatibility for those who are
// using Parallel.For(Each), since we aren't changing the TaskScheduler.Current.
var oldSyncContext = SynchronizationContext.Current;
try
{
using (var syncContext = new InlineSynchronizationContext())
using (var cancellationEvent = context.CancellationToken.ShutdownToken.GetCancellationEvent())
{
SynchronizationContext.SetSynchronizationContext(syncContext);
var result = InvokeSynchronously(method);
if (result == null) return null;
var task = getTaskFunc(result);
var asyncResult = (IAsyncResult)task;
var waitHandles = new[] { syncContext.WaitHandle, asyncResult.AsyncWaitHandle, cancellationEvent.WaitHandle };
while (!asyncResult.IsCompleted && WaitHandle.WaitAny(waitHandles) == 0)//這裏也同樣
{
var workItem = syncContext.Dequeue();
workItem.Item1(workItem.Item2);
}
return task.GetTaskLikeResult(result, method.ReturnType);
}
}
finally
{
SynchronizationContext.SetSynchronizationContext(oldSyncContext);
}
}
有問題的代碼就是
var result = scheduledTask.GetAwaiter().GetResult();
以及
while (!asyncResult.IsCompleted && WaitHandle.WaitAny(waitHandles) == 0)
截至目前1.8.22版本還是沒有解決這個問題,有人提了issue了,但是要到2.0.0版本才會解決。