Skip to content
This repository has been archived by the owner on Apr 15, 2020. It is now read-only.

Prevent sync operations from becoming async #36

Closed
yaacovCR opened this issue Jan 13, 2020 · 0 comments
Closed

Prevent sync operations from becoming async #36

yaacovCR opened this issue Jan 13, 2020 · 0 comments

Comments

@yaacovCR
Copy link
Owner

yaacovCR commented Jan 13, 2020

See #33 and https://github.com/apollographql/graphql-tools/pull/1057/files.

delegateToSchema should only return a promise when the configured executor/subscriber requires async work., e.g. not with execution via a local subschema.

Similarly, when not merging fields, handleObject should not convert lists of results to lists of promises to results,. Even when merging fields, given the above change, conversion to lists of promises to results should only be performed when necessary, i.e. when merging must be async.

yaacovCR added a commit that referenced this issue Jan 21, 2020
delegateToSchema now returns a synchronous result when possible rather
than always returning a promise.
merging types is also performed synchronously when possible.

Fixes #36.
yaacovCR added a commit that referenced this issue Jan 21, 2020
delegateToSchema now returns a synchronous result when possible rather
than always returning a promise.
merging types is also performed synchronously when possible.

Fixes #36.
yaacovCR added a commit that referenced this issue Feb 27, 2020
delegateToSchema now returns a synchronous result when possible rather
than always returning a promise.
merging types is also performed synchronously when possible.

Fixes #36.
yaacovCR added a commit that referenced this issue Mar 26, 2020
delegateToSchema now returns a synchronous result when possible rather
than always returning a promise.
merging types is also performed synchronously when possible.

Fixes #36.
yaacovCR added a commit that referenced this issue Mar 26, 2020
delegateToSchema now returns a synchronous result when possible rather
than always returning a promise.
merging types is also performed synchronously when possible.

Fixes #36.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant