-
Notifications
You must be signed in to change notification settings - Fork 97
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
Fix prefetching the same GQL type for batch request #658
Fix prefetching the same GQL type for batch request #658
Conversation
479c103
to
8c648a5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR @grynchuk! Everything looks pretty good - some CS fixes needed.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #658 +/- ##
============================================
- Coverage 95.72% 95.29% -0.44%
- Complexity 1773 1818 +45
============================================
Files 154 171 +17
Lines 4586 4841 +255
============================================
+ Hits 4390 4613 +223
- Misses 196 228 +32 ☔ View full report in Codecov by Sentry. |
@oojacoboo These fixes merged for the Would you approve PR with the same changes, but for |
@andrew-demb maybe it's just a better idea to test the changes in |
I have issue with getting wrong prefetch data while query in batch mode. In short, first query prefetch specific data and set it to prefetch buffer, second query use previous query prefetch buffer instead of using its own and get incomplete data set. I have to note that this issue is relates to prefetching of nested data, see test for mo details.
Request:
Response
Actual:
Expected: