Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not merge: Centralised caching via cacheExchange #98

Closed
wants to merge 17 commits into from

Conversation

kitten
Copy link
Member

@kitten kitten commented Jul 1, 2018

Please do not merge this into next as it's supposed to be outside of the 0.3 release's scope

This moves all caching logic to exchanges. To be more precise, when the cacheExchange now invalidates or updates the cache it does so via a client method. This client method also dispatches events to all components that can then check the cache keys they receive against the ones that they got from sending the last query operation.

Hence the components don't have to perform any cache logic themselves. They rely on receiving events from the client and that when executing a query operation the updated operation will go through the cache again, updating the component.

@kitten kitten force-pushed the feature/central-exchange-caching branch from f1f5885 to 0b6d832 Compare July 1, 2018 00:25
@coveralls
Copy link

coveralls commented Jul 1, 2018

Coverage Status

Coverage decreased (-1.6%) to 94.672% when pulling 3b9c2ed on feature/central-exchange-caching into 5e113c4 on next.

@kitten kitten force-pushed the feature/central-exchange-caching branch from 99fca57 to 3b9c2ed Compare July 1, 2018 01:17
@kitten kitten closed this Nov 19, 2018
@kitten kitten deleted the feature/central-exchange-caching branch February 14, 2019 03:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants