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

Several optimization to reduce outputobject size #307

Merged
merged 12 commits into from
Sep 10, 2021

Conversation

kleunen
Copy link
Contributor

@kleunen kleunen commented Aug 29, 2021

Several optimizations to reduce size of OutputObject:

  • Use OsmID instead of handle to lookup generated geometry (OsmID should be 42 bits now)
  • Use non-owning reference in tileindex and remove reference count from output object
  • Use intrusive pointer for attribute store reference
  • Prevent a temporary created in the mmap file in osm store

@kleunen kleunen changed the title Remove reference counted outputobject Remove reference count outputobject Aug 29, 2021
@kleunen kleunen changed the title Remove reference count outputobject Several optimization to reduce outputobject size Sep 5, 2021
@systemed systemed merged commit a9a54f9 into systemed:master Sep 10, 2021
@systemed
Copy link
Owner

10% reduction in total RAM used when converting Great Britain with no discernible impact on runtime - that's excellent! Thanks as ever.

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.

2 participants