Rename AsyncCacheCell to TaskCell - Githubissues

jet / equinox

.NET event sourcing library with CosmosDB, DynamoDB, EventStoreDB, message-db, SqlStreamStore and integration test backends. Focused at stream level; see https://github.com/jet/propulsion for cross-stream projections/subscriptions/reactions

https://github.com/jet/dotnet-templates

Apache License 2.0

472 stars 68 forks source link

Rename AsyncCacheCell to TaskCell #432

Closed bartelink closed 11 months ago

bartelink commented 11 months ago

I feel emphasizing that its a cell that manages a single in flight Task is better served by this name, i.e.:

yes it caches, but handles expiration too
half the point is that it doesn't cache failures
it is mainly Task based (though it supports Async)

But...

it uses an AsyncLazy internally, and is arguably an extension to that, which might suggest AsyncLazyCell as a better name
maybe AsyncLazy should also be renamed to LazyTask given how it deviates from the original AsyncLazy?

cc @eiriktsarpalis @nordfjord