【发布时间】:2021-02-14 04:02:15
【问题描述】:
我有一个 IAsyncEnumerable<string> 流,其中包含从 Web 下载的数据,我想将每条数据异步保存在 SQL 数据库中。所以我使用了来自System.Linq.Async 库的ForEachAwaitAsync 扩展方法。我的问题是下载和保存每条数据是按顺序进行的,而我希望它同时发生。
澄清一下,我不想同时下载多个数据,也不想同时保存多个数据。我想要的是,当我在数据库中保存一条数据时,下一条数据应该同时从网络上下载。
以下是我当前解决方案的最小(人为)示例。下载五个项目,然后将其保存在数据库中。下载每个项目需要 1 秒,保存则需要 1 秒:
async IAsyncEnumerable<string> GetDataFromWeb()
{
foreach (var item in Enumerable.Range(1, 5))
{
Console.WriteLine($"{DateTime.Now:HH:mm:ss.fff} > Downloading #{item}");
await Task.Delay(1000); // Simulate an I/O-bound operation
yield return item.ToString();
}
}
var stopwatch = Stopwatch.StartNew();
await GetDataFromWeb().ForEachAwaitAsync(async item =>
{
Console.WriteLine($"{DateTime.Now:HH:mm:ss.fff} > Saving #{item}");
await Task.Delay(1000); // Simulate an I/O-bound operation
});
Console.WriteLine($"Duration: {stopwatch.ElapsedMilliseconds:#,0} msec");
代码正在运行,但不是我想要的方式。总持续时间约为 10 秒,而不是理想的约 6 秒。
实际不受欢迎的输出:
04:55:50.526 > Downloading #1
04:55:51.595 > Saving #1
04:55:52.598 > Downloading #2
04:55:53.609 > Saving #2
04:55:54.615 > Downloading #3
04:55:55.616 > Saving #3
04:55:56.617 > Downloading #4
04:55:57.619 > Saving #4
04:55:58.621 > Downloading #5
04:55:59.622 > Saving #5
Duration: 10,115 msec
假设的理想输出:
04:55:50.000 > Downloading #1
04:55:51.000 > Saving #1
04:55:51.000 > Downloading #2
04:55:52.000 > Saving #2
04:55:52.000 > Downloading #3
04:55:53.000 > Saving #3
04:55:53.000 > Downloading #4
04:55:54.000 > Saving #4
04:55:54.000 > Downloading #5
04:55:55.000 > Saving #5
Duration: 6,000 msec
我正在考虑实现一个名为ForEachConcurrentAsync 的自定义扩展方法,它与上述ForEachAwaitAsync 方法具有相同的签名,但其行为允许同时对项目进行枚举和操作。下面是这个方法的一个存根:
/// <summary>
/// Invokes and awaits an asynchronous action on each element in the source sequence.
/// Each action is awaited concurrently with fetching the sequence's next element.
/// </summary>
public static Task ForEachConcurrentAsync<T>(
this IAsyncEnumerable<T> source,
Func<T, Task> action,
CancellationToken cancellationToken = default)
{
// What to do?
}
如何实现这个功能?
附加要求:
- 在取消或失败的情况下泄漏正在运行的任务是不可接受的。当方法完成时,所有启动的任务都应该完成。
- 在枚举和操作都失败的极端情况下,应该只传播两个异常中的一个,并且任何一个都可以。
- 该方法应该是真正异步的,并且不应阻塞当前线程(除非
action参数包含阻塞代码,但这是调用者的责任来防止)。
说明:
-
如果保存数据的时间比从网络下载数据的时间长,该方法应该不继续提前下载更多项目。最多只能预先下载一份数据,而保存前一份数据。
-
带有网络数据的
IAsyncEnumerable<string>是这个问题的起点。我不想更改IAsyncEnumerable<string>的生成器方法。我想对它的元素采取行动(通过将它们保存到数据库中),同时枚举可枚举的元素。
【问题讨论】:
-
您需要从
action调用中收集任务并在最后执行Task.WhenAll。 -
@juharr 理想情况下,我希望避免在枚举期间跟踪所有任务。
IAsyncEnumerable<T>理论上可以发出无限的元素。 -
如果你真的获得了无限的项目集合,那么代码将永远不会完成,但要处理它,你只需要一个缓冲区,一旦达到限制,你可以执行
WhenAny然后删除完成的任务。因为您可以在第一次保存之前完成所有下载,因此除非您想触发并忘记它们,否则无法在不以某种方式跟踪任务的情况下继续迭代集合。 -
@juharr 是的,保持有限的任务缓冲区当然是可能的。我不知道这将如何帮助我实现我想要的行为。关于“在第一次保存之前完成所有下载”,这不是我想要的功能。最多只能下载一条数据,保存前一条。
-
我理论上是说从集合中获取项目的时间可能远少于执行操作的时间,在这种情况下,我假设您希望按顺序而不是并行获取项目,但您希望这些操作不会延迟获取下一个项目。如果您说要等到第一个项目被保存后再下载第二个项目,那么您对需要多长时间的期望是完全错误的。
标签: c# concurrency async-await iasyncenumerable