graphql-java: OverlappingFieldsCanBeMerged is slow
OverlappingFieldsCanBeMerged can be very slow when many fragments in the query.
Scala implementation resolved it by optimizing the algorithm.
See https://github.com/sangria-graphql-org/sangria/pull/12.
Can we do something similar in graphql-java?
About this issue
- Original URL
- State: closed
- Created 4 years ago
- Comments: 16 (6 by maintainers)
Commits related to this issue
- adds experimental fast overlapping-fields-can-be-merged algorithm fixes #1786 — committed to redhead/graphql-java by deleted user 4 years ago
hi,
we are planning to address this issue via https://github.com/graphql-java/graphql-java/pull/2495
Special thanks to @redhead: we really appreciate the effort you put in, but ultimately we could not accept a scala version of the algorithm from a longterm maintainability POV. But we implemented the same algorithm based on the xing article.
We are planning to replace the current validation completely with the more performant one.
Andi