-
Notifications
You must be signed in to change notification settings - Fork 825
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
mapnik2 uses a lot more memory than before #657
Comments
[springmeyer] Hmm, possibilities are the new feature caching (#624) |
[jburgess777] If I force cache_features=false in apply_to_layer() then the very high peak memory usage no longer occurs. I'll try to delve into the caching code a little to see if there are excessive objects being cached or if the objects themselves appear bloated. |
[springmeyer] artem on skype mentioned with the mq style he only got ~180 MB max, so this style must not read nearly the same amount of data. I ask him for the bbox and tile so I can test on a system without mod_tile:
|
[springmeyer] assigning to mapnik2 release, milestone, as this needs further addressing before release. See original ticket at #624 |
[springmeyer] r2636 disables feature caching, but enables it to be turned on as an option on a layer. So, experienced users that want to test th potential io optimization for rendering layers with multiple styles (that do not pull enormous amounts of data) can turn on feature caching. |
When I render the following tile with mod_tile's renderd it requires significantly more memory when I use the trunk mapnik2 code instead of the mapnik-0.7.x branch.
http://tile.osm.org/10/511/340.png
-- this meta tile covers all of central London and hence it is a pretty difficult test case.
With mapnik-0.7.x the peak virtual memory usage of the renderd process is around 1GB
With mapnik2 the peak usage is over 8GB:
The rendering time is a lot slower, probably due to the swapping caused by this extra memory usage.
I have not dug any deeper to figure out where the memory is being used. I'll probably start by turning off the new featureset caching because it looks like a likely culprit.
The text was updated successfully, but these errors were encountered: