-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Avoid full reapplication of cache-and-network
and network-only
fetch policies after successful fetchMore
#9504
Avoid full reapplication of cache-and-network
and network-only
fetch policies after successful fetchMore
#9504
Commits on Mar 29, 2022
-
Adjust fetchMore tests to use TypedDocumentNode and subscribeAndCount.
Although these test updates may seem substantial, I believe this refactoring makes the tests more robust without changing what they test. To that end, it's important to note these tests are all passing at this point in the commit history, before any of the more substantive changes from PR #9504, and continue passing even after those changes are introduced, with relatively few additional test changes.
Configuration menu - View commit details
-
Copy full SHA for 8f7a3ed - Browse repository at this point
Copy the full SHA 8f7a3edView commit details -
Configuration menu - View commit details
-
Copy full SHA for c274f68 - Browse repository at this point
Copy the full SHA c274f68View commit details -
Configuration menu - View commit details
-
Copy full SHA for a6ba39c - Browse repository at this point
Copy the full SHA a6ba39cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 7e3dd93 - Browse repository at this point
Copy the full SHA 7e3dd93View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6177575 - Browse repository at this point
Copy the full SHA 6177575View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6dccb8d - Browse repository at this point
Copy the full SHA 6dccb8dView commit details -
Configuration menu - View commit details
-
Copy full SHA for a1d4a82 - Browse repository at this point
Copy the full SHA a1d4a82View commit details -
Configuration menu - View commit details
-
Copy full SHA for 53a246b - Browse repository at this point
Copy the full SHA 53a246bView commit details -
Configuration menu - View commit details
-
Copy full SHA for e320eb5 - Browse repository at this point
Copy the full SHA e320eb5View commit details -
Configuration menu - View commit details
-
Copy full SHA for c5b522d - Browse repository at this point
Copy the full SHA c5b522dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 3f7bce4 - Browse repository at this point
Copy the full SHA 3f7bce4View commit details -
Configuration menu - View commit details
-
Copy full SHA for 857583c - Browse repository at this point
Copy the full SHA 857583cView commit details -
Remove nagging updateQuery deprecation warning.
Now that fetchMore's updateQuery callback is implemented in terms of the supported/documented cache.updateQuery method, I feel better about allowing fetchMore to continue to take an updateQuery callback. Also, everyone with any ability to migrate from updateQuery to InMemoryCache field policies has presumably already done so, so this warning is less useful now than it was following the release of AC3.
Configuration menu - View commit details
-
Copy full SHA for 1b3406a - Browse repository at this point
Copy the full SHA 1b3406aView commit details -
Configuration menu - View commit details
-
Copy full SHA for f7dd5f4 - Browse repository at this point
Copy the full SHA f7dd5f4View commit details -
Use reobserveCacheFirst in !diff.complete case, too.
When `!diff.complete`, `oq.reobserveCacheFirst()` should behave exactly like `oq.reobserve()`, since the fetch policies `reobserveCacheFirst` modifies (`network-only` and `cache-and-network`) behave the same as the `cache-first` policy when cache results are incomplete.
Configuration menu - View commit details
-
Copy full SHA for 33e85c7 - Browse repository at this point
Copy the full SHA 33e85c7View commit details -
Configuration menu - View commit details
-
Copy full SHA for 359ed59 - Browse repository at this point
Copy the full SHA 359ed59View commit details -
Configuration menu - View commit details
-
Copy full SHA for 521ec2d - Browse repository at this point
Copy the full SHA 521ec2dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 0548f29 - Browse repository at this point
Copy the full SHA 0548f29View commit details -
Configuration menu - View commit details
-
Copy full SHA for cfc8cad - Browse repository at this point
Copy the full SHA cfc8cadView commit details -
Bring back cache.batch to ensure delivery of unchanged results.
This reverts commits d5463be and 0170f32, with a new test showing why the backup reobserveCacheFirst call in the finally block is important: sometimes the cache write doesn't change any data in the cache, so no broadcast happens, but we still need to deliver the final loading:false result for the fetchMore request.
Configuration menu - View commit details
-
Copy full SHA for e5ed91a - Browse repository at this point
Copy the full SHA e5ed91aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 5036dc5 - Browse repository at this point
Copy the full SHA 5036dc5View commit details