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

gateway: use go-micro store to cache #3250

Merged
merged 9 commits into from
Sep 23, 2022
Merged

gateway: use go-micro store to cache #3250

merged 9 commits into from
Sep 23, 2022

Conversation

butonic
Copy link
Contributor

@butonic butonic commented Sep 19, 2022

This PR replaces the in memory caches of the gateway with the go-micro store interface. It allows sharing the cache using redis or etcd for redundant deployments and the correct timing of stat cache invalidation because the dataprovider can now invalidate the cache when the upload has finished.

first step for owncloud/ocis#4427

  • allow datagateway to clean cache when file is available

@butonic butonic added the feature New feature label Sep 19, 2022
@butonic butonic self-assigned this Sep 19, 2022
@update-docs
Copy link

update-docs bot commented Sep 19, 2022

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@butonic butonic force-pushed the micro-cache-store branch 2 times, most recently from 7435358 to 14e8d58 Compare September 19, 2022 15:10
Copy link
Contributor

@aduffeck aduffeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome work. LGTM except for one issue with invalidating the stat cache with tus uploads.

pkg/rhttp/datatx/manager/tus/tus.go Outdated Show resolved Hide resolved
@butonic butonic force-pushed the micro-cache-store branch 2 times, most recently from 0f2c78c to 17181b9 Compare September 23, 2022 14:27
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants