search

botobag / artemis

GraphQL implementation for Go
Other
10 stars 0 forks source link

[graphql] Add LoadManyWith for DataLoaderManagerBase. #124

Closed zonr closed 5 years ago

zonr commented 5 years ago

This method makes use of dataloader.LoadMany to save call overheads when using dataloader.Load for each key. It also makes pendingLoaders in DataLoaderManagerBase be updated once with regardless of number of keys.

codecov[bot] commented 5 years ago

Codecov Report

Merging #124 into master will increase coverage by <.01%. The diff coverage is 78.57%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #124      +/-   ##
==========================================
+ Coverage   73.81%   73.82%   +<.01%     
==========================================
  Files          66       66              
  Lines        6957     6971      +14     
==========================================
+ Hits         5135     5146      +11     
- Misses       1554     1556       +2     
- Partials      268      269       +1
Impacted Files Coverage Δ
graphql/resolve_info.go 84.61% <78.57%> (-1.66%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f1666d9...35bee57. Read the comment docs.