merlinnot / nominatim-docker

Fully-featured container for geocoding, reverse geocoding and address lookup based on Nominatim and Open Street Map data.
MIT License
55 stars 35 forks source link

Feature requirements for v2 #15

Open merlinnot opened 5 years ago

merlinnot commented 5 years ago

I intend to work on a v2 version and I'd love to hear your feedback and brainstorm some ideas.

What I have in mind is:

ryan-morris commented 5 years ago

I tend to avoid using databases inside docker containers because they are too difficult to tune properly, especially with one as big as OSM. Though I do understand this is a necessity for most people to just get up and running.

I have a cluster of postgres servers I'd love to be able to offload to. If possible it would be great if the API just had an environment variable for pointing to a database, then link them in the default compose configuration. The problem with "external" databases from nominatim is the required extension that gets installed, I'm not sure if this is something the API container could handle during intial setup or something you were planning on addressing in the postgres container image. Also, if using something like docker swarm, there might be more than 1 instance of the API container running, so preventing conflicts or issues during updates may be problematic.

bryanrideshark commented 4 years ago

If anything, it would be incredibly helpful just to separate it into two containers.

UntitleDude commented 4 years ago

I'd love to be able to initialize the build blocks of the containers until you have to import data. Having to re-initialize the first steps can take quite a lot of time. Take a look at this other nominatim-docker which uses this mechanic.