graphile / graphile/crystal

v5: A second polymorphism optimization opportunity

Open
#2,513 0 comments 0 reactions 0 assignees View on GitHub
✨ feature
Dominant language
TypeScript
Stars
12.9k
Forks
625
Avg merge
5h 23m
Merged PRs (30d)
24

Description

ref: `grafast/grafast/__tests__/dcc/queries/item-lootbox-minimum.test.graphql`

Review the plan diagram and experiment with a new way of handling the polymorphism here.

Lootbox is not polymorphic but Item is.

Currently the plan has more parallel paths than it should as many of the paths could theoretically be collapsed down or deduplicated such that only one path is executed, but not using the current way of deduplication due to the way the schema is written.

![Image](https://github.com/user-attachments/assets/35ea9ca7-5aa5-4556-a0d8-e627e8ddcc22)
![Image](https://github.com/user-attachments/assets/3d9b7ac6-9361-4740-b644-5be09931c61a)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.