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

Include all metadata for locations #25

Open
CloudNiner opened this issue Aug 13, 2020 · 0 comments
Open

Include all metadata for locations #25

CloudNiner opened this issue Aug 13, 2020 · 0 comments

Comments

@CloudNiner
Copy link
Contributor

CloudNiner commented Aug 13, 2020

As discussed in #12 not all use cases require all metadata for locations and it has a high storage cost. There are some use cases though, like generating complete augmented diffs, that require all metadata. Keeping all metadata within OSMExpress as opposed to some external store is one option that reduces operational complexity at the cost of significant extra storage. It would be useful to hear what other use cases for all metadata might exist.

Summary of a brief discussion I had with @bdon regarding potential implementations:

  1. Are we sure we need username in addition to uid? Username, as a variable length string field, would be the most painful to add.

  2. This could be implemented as a "complete metadata" option, although in that case all other commands would need to know if the database was created in complete or slim (without location metadata) mode. In complete mode the locations table is not used and the nodes table would contain location, tags and metadata. In Slim mode the locations table is used and the nodes table only stores tags. We'd need to ensure that complete mode code does not impact performance of slim mode.

This task doesn't depend on #1, but may be simpler or use less storage space if it is addressed first.

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

1 participant