Unions that list each other fail with a StackOverflowError
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 55/100
Piste de recherche
Start by reading SchemaParser#getLeafUnionObjects and trace how nested union members are expanded; the issue identifies the recursive expansion as the source of the StackOverflowError. Reproduce with the mutual and self-referencing schemas shown in the issue, including a field that returns the inner union. Done when those schemas no longer overflow and the invalid union membership is handled consistently with the chosen behavior.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Two unions that list each other fail the schema build with a StackOverflowError:
type Query {
results: [Result!]!
}
union Result = Book | Media
union Media = Movie | Result
Same for a union that lists itself (union Result = Book | Result).
SchemaParser#getLeafUnionObjects expands nested unions recursively and doesn't keep track of the ones it already expanded. On master this only happens when some field returns the inner union, or for the self-member case. Otherwise the scanner fails first with "No object type found". After #866 the scanner gets past that, so the first schema above overflows too.
The spec doesn't allow unions as members at all, so I think either flattening each union once (both would be Book | Movie) or failing with an error that names the unions is fine. Probably the error, since nobody asked for cycles.
- Langage dominant
- Kotlin
- Étoiles
- 824
- Forks
- 175
- Merge moyen
- 2 j 5 h
- PR mergées (30 j)
- 52
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de graphql-java-kickstart/graphql-java-tools
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
graphql-java-kickstart/graphql-java-tools#893 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 3/5 1-2 jours Accessibilité débutants 74/100
graphql-java-kickstart/graphql-java-tools#896 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 72/100
graphql-java-kickstart/graphql-java-tools#889 ·
Les mainteneurs répondent en général sous 1 jour
-
Reject directives applied at the wrong location, without a required argument or twice in 15.0.0Ouverteenhancement
Difficulté 3/5 1-2 jours Accessibilité débutants 72/100
graphql-java-kickstart/graphql-java-tools#884 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
graphql-java-kickstart/graphql-java-tools#881 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de graphql-java-kickstart/graphql-java-tools
Issues similaires
-
enhancement impact:U1 P3 reach:R2 size:S
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 73/100
ankidroid/Anki-Android#22677 ·
Les mainteneurs répondent en général sous 1 jour
-
Type Pick Request
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
reactwg/react-native-releases#1456 · 1 commentaire ·
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
yschimke/homeassistant-remotecompose#1038 ·
Les mainteneurs répondent en général sous 1 jour
-
area:data-quality importer size:S
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
enorm-labs/event-junkie#3707 ·
Les mainteneurs répondent en général sous 1 jour