@connection directive support

[bennyhobart]

Hi guys, I'm not sure if this is an issue with my application but it seems like the @connection directive isn't working as expected

the two places I use it are as such, (code stripped for brevity)

activities(first: 30) @connection(key: "activities")
activities(first: 30, after: $cursor) @connection(key: "activities") 

If I remove the cursor from the second query it is working fine, any ideas?

[nevir]

Ah, hmm, this cache doesn't have explicit support for @connection—though, IIRC, most of that should be handled by Apollo client explicitly.

I'm curious: what keys end up being written to when you load the first query as opposed to the second? Try turning on verbose mode, which'll log out snapshots for each write and lists of ids that have changed; might help us narrow it down

[bennyhobart]

@neviri picked this out from the logs, it looks like they are being stored in two different locations

this is the root log

this is the important part

Any action I can take to help?

[bennyhobart]

I think at this point

apollo-cache-hermes/src/operations/SnapshotEditor.ts

Line 279 in 5e2b760

if (node.args) {

We determine if the field is a connection directive? and change the key based on that

[nevir]

Yeah, or potentially earlier when parsing the query (ParsedQueryNode) - however, we also have to teach the cache how to combine the two lists too (maybe append by default, but that quickly falls over), right?

[nevir]

Re:

Any action I can take to help?

We're unfortunately not going to have much bandwidth to dig into this much for the next couple weeks—if you want to take a stab at an implementation, that'd be a huge help! It might also be worth checking out how inmemory handles the @connection directive

[bennyhobart]

I'll take a stab when I get the opportunity, we should avoid merging there isn't enough information to do so, overwrite the previous result with the new which will keep inserts fast, it's the responsibility of the application developer — when fetching more) — to maintain this state, thanks for the research into in-memory cache's implementation