Skip to content
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

Fix a bug where changes to GWC layers won't be propagated to other pods. #570

Merged
merged 1 commit into from
Nov 3, 2024

Conversation

groldan
Copy link
Member

@groldan groldan commented Nov 3, 2024

In vanilla GeoServer, CatalogConfiguration is the TileLayerConfiguration contributed to the app context to serve TileLayers (GeoServerTileLayer) from the GeoServer Catalog by means of a TileLayerCatalog.

This update contributes a different TileLayerConfiguration for the same purpose, GeoServerTileLayerConfiguration, which is a distributed-event-aware decorator over the actual CloudCatalogConfiguration implementation of TileLayerCatalog.

Since CloudCatalogConfiguration is therefore not a Spring bean (to avoid registering it as a delegate to TileLayerDispatcher), TileLayerEvents need to be relayed from GeoServerTileLayerConfiguration to CloudCatalogConfiguration#onTileLayerEventEvict().

In vanilla GeoServer, `CatalogConfiguration` is the `TileLayerConfiguration`
contributed to the app context to serve `TileLayer`s (`GeoServerTileLayer`)
from the GeoServer `Catalog` by means of a `TileLayerCatalog`.

This update contributes a different `TileLayerConfiguration` for the same purpose,
`GeoServerTileLayerConfiguration`, which is a distributed-event-aware decorator
over the actual `CloudCatalogConfiguration` implementation of `TileLayerCatalog`.

Since `CloudCatalogConfiguration` is therefore not a Spring bean (to avoid
registering it as a delegate to `TileLayerDispatcher`), `TileLayerEvents` need
to be relayed from `GeoServerTileLayerConfiguration` to
`CloudCatalogConfiguration#onTileLayerEventEvict()`.
@groldan groldan force-pushed the bug/gwc_layer_config_propagation branch from 4858dac to f4dfa54 Compare November 3, 2024 07:59
@groldan groldan merged commit 48b6d30 into geoserver:main Nov 3, 2024
3 checks passed
@groldan groldan deleted the bug/gwc_layer_config_propagation branch November 3, 2024 14:07
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