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

Upgrade to federation-jvm 5.0.0 #972

Closed
wants to merge 1 commit into from

Conversation

dariuszkuc
Copy link
Contributor

Federation JVM v5 is now using graphql-java v22 which had following breaking changes in behavior

  • Apollo CacheControlInstrumentation - was using old deprecated instrumentation methods so old version will not work with latest spring-graphql version
  • there was a breaking change in the schema printing behavior of @deprecated information - previously it was always included regardless whether printed schema was filtering @deprecated directive definition. This is an issue for folks relying on self-managed Federation that uses _service { sdl } endpoint to obtain subgraph schemas as the supergraph schema would no longer include any deprecation information.

Federation JVM v5 is now using `graphql-java` v22 which caused following breaking changes in behavior

- Apollo `CacheControlInstrumentation` - was using old deprecated instrumentation methods so old version will not work with latest `spring-graphql` version
- there was a breaking change in the schema printing behavior of `@deprecated` information - previously it was always included regardless whether printed schema was filtering  `@deprecated` directive definition. This is an issue for folks relying on self-managed Federation that uses `_service { sdl }` endpoint to obtain subgraph schemas as the supergraph schema would no longer include any deprecation information.
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label May 17, 2024
@rstoyanchev rstoyanchev added type: dependency-upgrade A dependency upgrade and removed status: waiting-for-triage An issue we've not yet triaged labels May 18, 2024
@rstoyanchev rstoyanchev changed the title chore: update federation-jvm to v5.0.0 Update federation-jvm to v5.0.0 May 18, 2024
@rstoyanchev rstoyanchev added this to the 1.3.0 milestone May 18, 2024
@rstoyanchev rstoyanchev self-assigned this May 20, 2024
@dariuszkuc dariuszkuc deleted the patch-1 branch May 20, 2024 17:42
@rstoyanchev rstoyanchev changed the title Update federation-jvm to v5.0.0 Upgrade to federation-jvm 5.0.0 May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: dependency-upgrade A dependency upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants