-
-
Notifications
You must be signed in to change notification settings - Fork 18
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
Migrate from bundles to sqlite databases #105
Comments
related: pelias/wof-admin-lookup#249 |
I'll take this task |
loading planet via filesystem (old bundle data):
loading planet via filesystem (current data extracted from sqlite databases):
loading planet via sqlite databases (current data):
|
here's another benchmark of the filesystem (current data extracted from sqlite databases)
|
SummaryLooking exclusively at the Using the This is a huge difference, I wanted to know if it was due to the loading method or the data, so I deleted the bundle data and extracted the sqlite data to disk in the same directory structure as expected by the On a cold start the I tried running the It looks like work from the WOF team over the last ~6 months to add more data and improve geometries has negatively impacted the start-time performance, once the distributions are fixed and new bundles are created we can expect a 7x slowdown on start times using the I'd suggest we switch over to cc @orangejulius, @Joxit |
The SQLite supports is definitely a good move for pelias services 😄 |
I'd say this is done. We have been using the sqlite option in production for geocode.earth for some time. It does load a bit slower, and I notice that the workers are no longer at 100% CPU while loading, so there may be some more optimizations we can make. However, overall the process is much more stable, and having a single file instead of millions actually dramatically improves stability overall. |
My PIP service took 1347.156 seconds to star. below are the logs Even after this I am seeing lot of missing locality data like below- My main concern is /reverse api doesnt return any locality/localadmin/postalcode details in the response. even though i do have admin,venue,postalcode data for whosonfirst [sqlite bundles] Could you please let me know is there any reason that /v1/search API return locality/localadmin/postalcodes in the response but /v1/reverese doesnt.? |
Hi, I initially thought it was good to comment here as the issue was closely related to this. But later i thought i should open an new tkt.. Thanks for your support and kind help! |
PR pelias/whosonfirst#417 was recently merged which will allow the
pip-service
to load its data directly from sqlite databases instead of reading from disk.Before we make this change in production its important that we test the start times of a full planet import and compare the two loading strategies.
If loading from sqlite is satisfactorily fast then we should go ahead and migrate :)
The text was updated successfully, but these errors were encountered: