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

Writing to cache/store after generation #87

Closed
tomass opened this issue Apr 15, 2017 · 4 comments
Closed

Writing to cache/store after generation #87

tomass opened this issue Apr 15, 2017 · 4 comments

Comments

@tomass
Copy link

tomass commented Apr 15, 2017

Thank you for the great software!

Do I understand correctly, that at this time it is not possible to write vtile generated by tileserver on demand to cache/store. So that if the same request from the client comes in after some time, same tile would not have to be generated again.

Looks like currently cache/store is only filled by running tilequeue script and generating specified tiles (and it is hard to know which tiles would be requested, very low percentage of high zoom level tiles are ever requested so there is no point to generate them all and it takes a lot of time). Tiles not found in cache/store and generated on demand by tileserver are thrown away?

Thank you

@iandees
Copy link
Member

iandees commented Apr 15, 2017

I took out the store code in #85 because Mapzen is trying to optimize its tile response time (by doing more work in tilequeue and relying less on tileserver).

We probably should have kept it in there and configured it to be off on our servers. I'll look at doing that next week, but for now if you want to take a look at that pull request to see what was taken out you can see how we implemented storing tiles in tileserver.

@rmarianski
Copy link
Member

Like @iandees mentioned, We're planning on adding some kind of caching back in ourselves. Sounds like we might want to think about growing an abstraction for this, or maybe use the same store interface that exists but grow a redis implementation for it.

@nvkelso
Copy link
Member

nvkelso commented Apr 16, 2017

@tomass In the meantime, I suggest using the tagged production release instead: https://github.com/tilezen/tileserver/releases/tag/v1.4.0.

@tomass
Copy link
Author

tomass commented Apr 16, 2017

Thank you a lot for information and suggestions! It was really helpful!

@tomass tomass closed this as completed Apr 16, 2017
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

No branches or pull requests

4 participants