additional addendum fields

pelias / openstreetmap

Import pipeline for OSM in to Pelias

MIT License

112 stars 72 forks source link

additional addendum fields #529

Closed missinglink closed 4 years ago

missinglink commented 4 years ago

following on from https://github.com/pelias/openstreetmap/pull/528 this PR uncomments the three fields I wanted to enable in the past but wasn't sure how much additional disk space would be required.

turns out that addendum fields don't add a lot of bytes.

I also took the liberty to modernize the codebase a bit and fix a test case that wasn't being run due to using a duplicate variable name.

missinglink commented 4 years ago

Oh man, I must really want this feature, I did the PR twice!

missinglink commented 4 years ago

I cherry-picked the additional tests from the other PR

missinglink commented 4 years ago

closes #494