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

Remove ICache #389

Closed bartelink closed 1 year ago

bartelink commented 1 year ago

Spike - it could be this much shorter if we removed ICache ;) @DSilence

bartelink commented 1 year ago

Yeah, @DSilence put it in for a reason though - the company he was working for was using another cache (which had Task interfaces which is why there was egregious Asyncness in the interface). But it was the V3 rollover last time I asked him, so maybe he'll be all YOLO on it for V4 :D

bartelink commented 1 year ago

@DSilence Thinking I'll bite the bullet on this as it makes the code more legible and removes a concept - if the system in question is upgrading, we'll hopefully be able to work out a solution without impeding that (i.e. open to reintroducing at some point, but taking the lack of traffic here as assent!)