Skip to content

BREAKING CHANGE: Make the KotlinDataFetcherFactoryProvider nullable #623

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

Merged
merged 1 commit into from
Feb 27, 2020

Conversation

smyrick
Copy link
Contributor

@smyrick smyrick commented Feb 27, 2020

📝 Description

We made the FunctionDataFetcher return nullable in this change #613. That is a good change but I only realized after integrating into our internal SDK that this also impacts the KotlinDataFetcherFactoryProvider.

KotlinDataFetcherFactoryProvider returns a DataFetcherFactory, and in graphql-java those are supposed to implement the same types that the DataFetcher returns. So this change fixed that to match the same types in graphql-java.

This is a breaking change as the interface signature has now been changed.

🔗 Related Issues

Data Fetcher returns nullable here: #613

We made the DataFetcher nullable in this change ExpediaGroup#613, so this lines up to match the same types in graphql-java
@smyrick smyrick added type: bug Something isn't working changes: major Changes require a major version labels Feb 27, 2020
@tapaderster
Copy link
Member

lgtm

@smyrick smyrick merged commit dd90c71 into ExpediaGroup:master Feb 27, 2020
@smyrick smyrick deleted the nullable-data-fetcher-provider branch February 27, 2020 18:51
dariuszkuc pushed a commit to dariuszkuc/graphql-kotlin that referenced this pull request Aug 5, 2022
We made the DataFetcher nullable in this change ExpediaGroup#613, so this lines up to match the same types in graphql-java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes: major Changes require a major version type: bug Something isn't working
Development

Successfully merging this pull request may close these issues.

2 participants