-
-
Notifications
You must be signed in to change notification settings - Fork 818
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
test #3594
test #3594
Conversation
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/theguild/graphql-tools/Ho5RTftMTyfeMpXfhf8WZeq8V2Cz |
The latest changes of this PR are not available as alpha, since there are no linked |
90f016b
to
35bdca8
Compare
Perhaps predictably, not so great performance wise. |
although not as much of a performance penalty as the other approach.... @ardatan any thoughts on these numbers? this approach could also help with defer/stream |
35bdca8
to
5ae17d5
Compare
5ae17d5
to
e40ad99
Compare
e40ad99
to
4fd214a
Compare
may be helpful if ExternalObject access were to be managed by a store -- in which case the store could be updated after initial result returned from original schema either with additional incremental delivery results or with results from other schemas
e619b78
to
c2146cc
Compare
4946aeb
to
4be9073
Compare
d34fb65
to
b8bf584
Compare
Closing this very stale pull request. |
these changes allow key and args functions to return promises, which may be helpful if ExternalObject access were to be managed by a store -- in which case the store could be updated even after initial result returned from original schema either with additional incremental delivery results or with results from other schemas.