PYR1-1801 Publish shapefile layers without an auto-published copy - #104
Merged
lambdatronic merged 1 commit intoSep 25, 2026
Merged
Conversation
Create the shapefile feature type with one POST under the store name, and drop the PUT to external.shp. That PUT made GeoServer auto-publish a feature type named after the shapefile; shapefiles sharing a basename (five isochrones.shp per fire, ELMFIRE and Pyretechnics per match-drop) raced for it, and the cleanup by name left isochrones1-4 behind in every workspace.
danielhvs
marked this pull request as ready for review
September 24, 2026 17:15
lambdatronic
approved these changes
Sep 25, 2026
lambdatronic
deleted the
PYR1-1801-publish-shapefiles-without-auto-publish
branch
September 25, 2026 16:12
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
Each shapefile store now publishes exactly one feature type, named after the store. Before this, every fire spread workspace ended up with
isochrones1toisochrones4next to the intendedelmfire_landfire_{30,50,70,90}_isochrones, and every match-drop workspace with 6isochrones_md-<id>_<ts>_<pct>1copies.Cause
:shapefilebranch sentPUT .../external.shp, and GeoServer auto-publishes a feature type named after the shapefile basename.external.shpPUTs run in parallel. Several shapefiles share a basename: 5 ×isochrones.shpper fire since PYR1-1579's rename, and ELMFIRE and Pyretechnics ship the sameisochrones_md-..._<pct>.shpper match-drop run. GeoServer suffixes all but the first.404 ...:isochroneson 09-23.Changes
:shapefilecreates the store, then POSTs a single feature type withname= store name andnativeName= shapefile basename (rest/create-feature-type-alias, the same request the rename step already sent). The auto-publishing PUT and the two cleanup DELETEs are gone.:geopackageis unchanged. Its table names are unique per store, so it has no collision.shapefile-layer-specs-testcovers the nested and flat cases and the style step.Existing leaked layers are not touched. Fire spread workspaces rotate out on their own; match-drop ones need a one-off cleanup per environment after this ships.
Verified on dev
I pinned the dev GeoSync UberJAR to
9565a00, removedfire-spread-forecast_wa-sisi_20260924_095400, and re-added it withherd trigger sync-fire-spread-forecast:elmfire_landfire_{10..90}_isochronesstore holds one feature type. There are noisochronesNcopies, and geoserver02's leak count dropped from 44 to 40 (wa-sisi is gone from the list)Workspace updated.with 0404 ...:isochronesnativeNameisochronesand SRS EPSG:32610, get real bboxes that widen from P10 to P90, render WMS GetMap200 image/png, and have GWC tile layers at[1, 1]with MVTelmfire_landfire_10_isochroneselmfire_landfire_30_isochroneselmfire_landfire_50_isochroneselmfire_landfire_70_isochroneselmfire_landfire_90_isochronesRelated Issues
Closes PYR1-1801
Relates to PYR1-1579, PYR1-1786