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

Adding global_synchronized_cache in valhalla_build_config #1851

Merged

Conversation

AurelienLP
Copy link
Contributor

Issue

#1850

Tasklist

  • Add tests
  • Review - you must request approval to merge any PR to master
  • Add #fixes with the issue number that this PR addresses
  • Generally use squash merge to rebase and clean comments before merging
  • Update the changelog
  • Update relevant documentation

@dnesbitt61
Copy link
Member

I might caution that I have been seeing some issues during data generation (valhalla_build_tiles) with some settings when "global_synchronized_cache": true is set in the config. I do not recommend having this set to true during the tile generation process.

@AurelienLP
Copy link
Contributor Author

Hi @dnesbitt61,

I've been experiencing this for the last couple weeks indeed. I think that trying to fix this will be a lot tougher than just not using it the tile generation process.

Thank you very much for this valuable warning!

@kevinkreiser kevinkreiser merged commit 7402cb8 into valhalla:master Jun 26, 2019
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

3 participants