★ wanayoo — archive 1999 https://github.com/graphql-java/graphql-java/pull/1668Nouvelle recherche | Portail wanayoo
Skip to content

/ graphql-java

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

Jump to bottom

Enhanced execution input (and hence data loader registry) is now captured before use #1668

Merged
merged 4 commits into from Jan 13, 2020
Merged

Enhanced execution input (and hence data loader registry) is now captured before use #1668

merged 4 commits into from Jan 13, 2020

Conversation

@bbakerman
Copy link
Member

@bbakerman bbakerman commented Oct 6, 2019 •
edited

See #1667

If a Instrumentation enhances the ExecutionInput and hence the DataLoaderRegistry instance, we can now use the right instance during dispatching
@bbakerman
Enhanced execution input (and hence data loader registry) is now capt…
Loading status checks…
26e7547 
…ured before use
@bbakerman bbakerman added this to the 14.0 milestone Oct 6, 2019
@bbakerman bbakerman self-assigned this Oct 6, 2019
bbakerman added 3 commits Jan 13, 2020
@bbakerman
merged master
Loading status checks…
bac51ad
@bbakerman
Fixed up the no data loader logic
Loading status checks…
6571cab
@bbakerman
Slight code improvement
Loading status checks…
abf7048
@bbakerman bbakerman merged commit 50c242d into graphql-java:master Jan 13, 2020
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@andimarek andimarek mentioned this pull request Jan 14, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@bbakerman bbakerman

Labels
None yet
Projects
None yet
Milestone
14.0
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
@bbakerman
You can’t perform that action at this time.