First of all, both your approaches are wrong, because they don't save you any requests (though the second one at least saves you time).
Your first code (the one with await
) does this:
- Make the request.
- Wait for the request to complete.
- If there already was a result in the cache, ignore the result of the request.
Your second code removes step 2, so it's faster, but you're still making lots of unnecessary requests.
What you should do instead is to use the overload of GetOrAdd()
that takes a delegate:
public Task<String> GetStuffAsync(String url)
{
return _cache.GetOrAdd(url, GetStuffInternalAsync);
}
This doesn't completely eliminate the possibility of requests that are ignored, but it does make them much less likely. (For that, you could try canceling requests that you know are being ignored, but I don't think that's worth the effort here.)
Now to your actual question. What I think you should do is to use the AddOrUpdate()
method. If the value isn't there yet, add it. If it's there, replace it if it's faulted:
public Task<String> GetStuffAsync(String url)
{
return _cache.AddOrUpdate(
url, GetStuffInternalAsync, (u, task) =>
{
if (task.IsCanceled || task.IsFaulted)
return GetStuffInternalAsync(u);
return task;
});
}