-
-
Notifications
You must be signed in to change notification settings - Fork 44
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
port download code to use the batch endpoint #513
base: master
Are you sure you want to change the base?
Conversation
This is ready for testing, there are some features I would have liked to add but will have to wait for a subsequent PR:
I still need to clarify a couple things with the OA team:
|
LGTM 👍 for decompression on Maybe you can add a openaddresses/utils/download_filtered.js Lines 24 to 32 in 60256bd
|
Yeah good idea about the logging, I'll go through and make sure everything is logged, even if it's at the debug log level, I also want to log the output of these child process shell commands if anything goes wrong. I have a chat going with the OA team at the moment to clarify some of the final questions:
|
Hi @Joxit, @missinglink, Any update on this? Is that planned to make this available in the pelias/docker repo? Thanks! |
…70a414af043ef95f See pelias#513 for a proper fix. Really we can just omit this commit, there's no need for the latest version, but it seems nice to try to keep all the pelias repos on the same config release.
this
draftPR updates the OA downloader to use thebatch.openaddresses.io
endpoint.a few things worth mentioning:
gzip
compressed rather thanzip
I haven't done any testing of the import process, only the download process so far