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

Remove unused catalog-service microservice and client #387

Merged
merged 1 commit into from
Dec 6, 2023

Conversation

groldan
Copy link
Member

@groldan groldan commented Dec 6, 2023

The catalog service has been out of commission for a long time now, becoming dead code.

The idea is still sound but the complexity of WebFlux makes it a maintenance nightmare. Plus, with Java 21's virtual threads, it will be possible to write a non WebFlux implementation with almost the same benefits, except maybe for the reactive back-pressure support.

@groldan groldan added build Issues related to the build and/or CI dependencies dependency management docker Issues related to docker images or docker composition QA labels Dec 6, 2023
The catalog service has been out of commission for a long time now,
becoming dead code.

The idea is still sound but the complexity of WebFlux makes it a
maintenance nightmare. Plus, with Java 21's virtual threads, it will be
possible to write a non WebFlux implementation with almost the same
benefits, except maybe for the reactive back-pressure support.
@groldan groldan force-pushed the remove_unsued_catalog_microservice branch from 23cabb9 to 42db253 Compare December 6, 2023 02:57
@groldan groldan merged commit 63a227b into geoserver:main Dec 6, 2023
2 checks passed
@groldan groldan deleted the remove_unsued_catalog_microservice branch December 6, 2023 03:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Issues related to the build and/or CI dependencies dependency management docker Issues related to docker images or docker composition QA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant