Skip to content

Feat/expire 4326 - #2

Open
eyalr1100 wants to merge 2 commits into
chore/sync-forkfrom
feat/expire-4326
Open

eyalr1100 wants to merge 2 commits into
chore/sync-forkfrom
feat/expire-4326

Conversation

@eyalr1100

Copy link
Copy Markdown

No description provided.

from_polygon_area() computed its tile row range with inline Web Mercator
arithmetic on the raw envelope, bypassing the projection's tile
conversion that every other expiry path uses. For Web Mercator the
conversion is the identity, so this is a no-op; for other projections it
stops both bounds collapsing to the same row, which silently skipped
polygon interiors.

Refs: MAPCO-11657
The command-line expire options only picked up tables whose geometry
column is Web Mercator, so a flex config storing EPSG:4326 produced an
empty expire list. Expiry now takes the SRID from the geometry columns
being expired, builds the tile conversion with osm2pgsql's own
reprojection for it, and refuses to start if those columns disagree.
The expire output stays in the Web Mercator tile scheme.

The SRID is stored on the expire output, so the main output and its
per-worker clones build their projection from the same value, at setup
and never per row. Lua-defined expire outputs keep the Web Mercator
default.

Refs: MAPCO-11657
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.

1 participant