【问题标题】:Task.WhenAll hangs out if there are no tasks to complete如果没有要完成的任务,Task.WhenAll 挂起
【发布时间】:2018-07-14 21:46:42
【问题描述】:

我有将对象列表插入 Mongo DB 的方法。

public class StorageService : IStorageService
{
  public Task<BulkWriteResult<Option>> SaveOptions(List<Option> contracts)
  {
    var context = new MongoContext<Option>();
    return context.SaveCollection(contracts);
  }
}

var optionIds = Task
  .WhenAll(storageService.SaveOptions(optionDetails.Values.ToList()))
  .Result;

如果合约列表为空,则没有对象可插入数据库,也没有任务可完成,因此 Task.WhenAll 会无限期地运行,从而造成死锁。

问题

如果列表为空,有没有办法返回空/已完成的任务,或者可能有更好的解决方案来获取插入结果,但同时正确处理没有结果的情况?

更新 #1

大概的结构。

WebApi - MVC 项目

[AcceptVerbs("POST")]
public Response<int> DownloadOptions([FromBody] ContractSelector data)
{
  .. some controller code

  var optionIds = Task
    .WhenAll(storageService.SaveOptions(optionDetails.Values.ToList()))
    .Result;

  // this method should gather data from multiple APIs
  // so I need Task.Result of all previous operations
  // I could make this method async and use await, but it's not the case here
}

类库项目,.NET 4.6.1

public class StorageService : IStorageService
{
  public Task<BulkWriteResult<Option>> SaveOptions(List<Option> contracts)
  {
    var context = new MongoContext<Option>();
    return context.SaveCollection(contracts);
  }
}

更新 #2

为什么异步/等待没有用。有 2 个外部 API 调用,一个是获取有关某些资产的一般信息,第二个是获取该资产的价格,我无法更改。因此,如果我想通过一种方法获取所有信息,我必须请求一般信息,然后等待等待结果,并根据一般信息请求相关价格。在此之后,我想将收集到的信息保存到数据库中,并在对我的 API 的响应中返回保存的 ID 列表。

调用顺序

1. UI
2. WebApi MVC Controller
3. Class Library
3.1 Request asset info - wait for the result
3.2 Get asset info - request prices for selected assets - wait for the result
3.3 Get asset info and prices info - save everything to DB - return response

var contracts = Task.WhenAll(optionService.GetContracts(params)).Result;
var prices = Task.WhenAll(optionService.GetOptionDetails(contracts)).Result;
var ids = Task.WhenAll(storageService.SaveOptions(prices.Values.ToList()));

因此,响应取决于 3.3,3.3 取决于 3.2,3.2 取决于 3.1。如果您知道如何将其全部转换为非阻塞呼叫,我会全力以赴。目前我认为 1 个 HTTP 请求中的 3 个阻塞调用优于 3 个单独的异步 HTTP 请求。

【问题讨论】:

  • 这里的快速测试表明Task.WhenAll 确实,实际上,当没有任务传入时停止...var nothingToDo = new List&lt;Task&gt;(); Task.WhenAll(nothingToDo); Console.WriteLine("All done"); - 所以...可能有这里的假设不正确。此外,您应该几乎从不使用.Result.Wait(),这种情况肯定比“几乎”更倾向于“从不”
  • 你只有一个任务,不管有没有,一个或多个合同,所以你甚至不需要WhenAll。话虽如此,就像 Marc Gravell 提到的那样,不要使用 Result 或 Wait,而是在需要等待任务/获取结果时使用 await。
  • optionIds在什么框架下运行?
  • 这个方法应该从几个 API 收集数据,所以我需要得到这个调用的结果,在这里我看不到 Result 的任何替代方法
  • ASP.NET Web API 针对异步代码进行了优化。为什么DownloadOptions 不是async Task&lt;Response&lt;int&gt;&gt; DownloadOptions?你不应该像这样用Result 阻塞线程。更重要的是,您不应该创建一个唯一目的是等待另一个TaskTask。充其量(阅读,worst),它应该只是storageService.SaveOptions(optionDetails.Values.ToList()).Result。但你真的应该只使用await storageService.SaveOptions(optionDetails.Values.ToList())

标签: c# mongodb asynchronous


【解决方案1】:

无法详细解释,但看起来问题是由 .NET 框架和 Mongo DB 驱动程序之间的不兼容引起的。目前我正在使用 .NET 4.6.1 和 Mongo 2.6.1,它工作正常。

昨天任何尝试将空的对象列表保存到 Mongo DB 导致死锁。

然后我尝试重构代码并安装最新版本的 Mongo DB,即 2.7.0。现在,任何使用 Mongo DB 的 CRUD 操作都会返回此错误。没有关于此异常的调用堆栈和更多详细信息。

MethodAccessException: Attempt by method 'MongoDB.Driver.ClientSession..ctor(MongoDB.Driver.IMongoClient, MongoDB.Driver.ClientSessionOptions, MongoDB.Driver.IServerSession, Boolean)' to access method 'MongoDB.Driver.Core.Clusters.ClusterClock..ctor()' failed.

然后我尝试将 Mongo DB 的版本降低到 2.6.1,当我尝试插入空的对象列表时,我得到了这个异常,这是正确的。当我插入至少一条记录时,它按预期工作。

System.AggregateException: 'One or more errors occurred.'
ArgumentException: Must contain at least 1 request.
Parameter name: requests

测试代码

var item = new Demo
{
  Symbol = "SomeSymbol",
  Expiration = "2019-01-01"
};

var list = new List<Demo>();

list.Add(item); 
list.Add(item); // if I comment these lines, then Mongo 2.6.1 returns correct exception

var optionIds = Task.WhenAll(storageService.Save(list)).Result;

保存方法作为Mongo repository pattern的一部分实现

public Task<BulkWriteResult<T>> SaveCollection(List<T> items)
{
  var records = new List<ReplaceOneModel<T>>();
  var processes = new List<Task<ReplaceOneResult>>();

  items.ForEach(contract =>
  {
    var record = new ReplaceOneModel<T>(Builders<T>
      .Filter
      .Where(o => o.Id == contract.Id), contract)
    {
      IsUpsert = true
    };

    records.Add(record);
  });

  return Collection.BulkWriteAsync(records);
}

【讨论】:

    猜你喜欢
    • 1970-01-01
    • 2016-01-30
    • 1970-01-01
    • 2014-09-21
    • 2021-07-17
    • 1970-01-01
    • 2020-05-22
    • 1970-01-01
    • 2021-11-29
    相关资源
    最近更新 更多