-
-
Notifications
You must be signed in to change notification settings - Fork 43
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
Store geometries for administrative areas #19
Comments
Still discussing ES vs disk vs S3 |
Current decision is to store in S3 and provide link in /place search. |
@thisisaaronland @heffergm: We're looking to re-serve the Who's on First geojson documents from S3 when someone requests the full details about a particular record that comes from Who's on First. Today, all documents come from our Elasticsearch index on the What might that setup look like? What kind of implications would there be for folks looking to use Who's on First from their own setup if we do this? |
Are we talking about serving the WOF record as an API response, or are you talking about something as a convenience for people to go look at the original geojson? I think you're going to end up having to reverse proxy the data from S3 via the API, it's all here: https://s3.amazonaws.com/whosonfirst.mapzen.com/ cc @baldur |
Likely as an API response (since we didn't opt to build a hypermedia-style I suppose if we're ok with non mapzenners requesting it directly from S3, On Thu, Jan 14, 2016 at 3:08 PM, Grant Heffernan notifications@github.com
David Riordan | Product Manager - Search | dave.riordan@mapzen.com | |
I think we need to maintain our response format which means we can't just serve the exact WOF record as-is. We'll need to fetch it from S3 and copy the parts we care about into our response object. The alternative to S3 access from API is storing the WOF data locally and reading it from disk when needed. This could be a pain at deploy time because we'd need to copy all of WOF to each API server. |
S3 access from inside AWS is quite fast... I'd be inclined to at least suggest starting there, rather than deal with local clones of a large dataset just to deploy the api. |
Yup, we're all on the same page then. The local clones of data would be plan B. Sounds like plan A is good, though. 🙌 |
Lets push this out of the milestone and approach it right afterwards. |
Hi everyone! Serving them only from the The |
First subset of #1: Storing geometries for admin areas in the document.
The text was updated successfully, but these errors were encountered: