Skip to content

Make queries per-table#110

Merged
rmarianski merged 1 commit intomasterfrom
query-by-table
Jul 31, 2017
Merged

Make queries per-table#110
rmarianski merged 1 commit intomasterfrom
query-by-table

Conversation

@rmarianski
Copy link
Copy Markdown
Member

@pnorman
Copy link
Copy Markdown
Contributor

pnorman commented Jul 29, 2017

404 error, could you provide some details on what it connects to?

@nvkelso
Copy link
Copy Markdown
Member

nvkelso commented Jul 29, 2017 via email

@zerebubuth
Copy link
Copy Markdown
Member

I've added more information to the README for raw tiles, which is still on a branch for the moment. That describes the overall goals and what we're trying to achieve, but it's also useful to know how we're getting there.

We've split the work up into several "phases", of which one is the tile-tasks issue linked to above (phase 2). The summary is:

  1. Phase 1 was about being able to calculate in Python all the min_zoom and feature properties that we're currently calculating in SQL (generated from YAML for the bonus confusement), and has now finished.
  2. Phase 2 is about querying the database on a per-table basis rather than a per-layer basis.
  3. Phase 3 is about generating flat files (RAW(R) tiles) for each table at zoom 10, and querying those instead of hitting the database.
  4. Phase 4 extends that to all zoom levels.

The main benefits (in more detail in that README) are that the PostgreSQL server is no longer on any "hot path", so doesn't ever become a bottleneck, and that we should be able to run multiple different styles/versions from the same infrastructure stack.

@rmarianski rmarianski changed the title WIP Make queries per-table Make queries per-table Jul 31, 2017
@rmarianski rmarianski merged commit d031f3a into master Jul 31, 2017
@rmarianski rmarianski deleted the query-by-table branch July 31, 2017 18:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants