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

Cache invalidation problems when database content changes #1578

Closed
mkovac-conductive opened this issue Nov 14, 2024 · 3 comments
Closed

Cache invalidation problems when database content changes #1578

mkovac-conductive opened this issue Nov 14, 2024 · 3 comments

Comments

@mkovac-conductive
Copy link

I would like to know what is the recommended way to handle tile cache invalidation when database content changes? Currently I need to restart martin server to get latest data on my Mapbox page.

My setup:

Frontend: React (with Mapbox)
Tileserver: Martin
Database: PostgreSQL

Thanks!

@CommanderStorm
Copy link
Collaborator

Currently, restarting is the only option.

While this should be pretty quick, this is not ideal (we know).
The current draft of an API route for this would be a massive performance hit and has thus not been merged.

See

@CommanderStorm
Copy link
Collaborator

I think we should track this in #288 instead, given that said issue already has a discussion about how this can/should be implemented

=> Closing as a duplicate of #288 ^^

@CommanderStorm CommanderStorm closed this as not planned Won't fix, can't repro, duplicate, stale Nov 15, 2024
@nyurik
Copy link
Member

nyurik commented Nov 15, 2024

I keep wondering if we should kill the tile cache for now - it is more of a nuicance than help - response caching can be done much better at the load balancer or reverse proxy level (e.g. GCP Load balancer / Varnish / Nginx / ...).

The only real reason for cache (why i initially introduced it) - was to cache pmtiles directory structure so it doesn't need to be refetched for each tile

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 a pull request may close this issue.

3 participants