You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently DefaultExecutionGraphQlService sets the DataLoaderRegistry in ExecutionInput to a new instance because it needs to make registrations, while the default instance in ExecutionInput is a no-op that raises exceptions.
We could improve this by using the DataLoaderRegistry from ExecutionInput provided that it's not the default no-op implementation, or has any registrations, which implies it's not the no-op. This would allow use of alternative DataLoaderRegistry like ScheduledDataLoaderRegistry.
The text was updated successfully, but these errors were encountered:
Currently
DefaultExecutionGraphQlService
sets theDataLoaderRegistry
inExecutionInput
to a new instance because it needs to make registrations, while the default instance inExecutionInput
is a no-op that raises exceptions.We could improve this by using the
DataLoaderRegistry
fromExecutionInput
provided that it's not the default no-op implementation, or has any registrations, which implies it's not the no-op. This would allow use of alternativeDataLoaderRegistry
likeScheduledDataLoaderRegistry
.The text was updated successfully, but these errors were encountered: