Repository navigation
unbreak LiveState unsub when references.size === 0 - #4832
subtleGradient wants to merge 5 commits into
Conversation
|
Thank you for this. Great catch. Would you mind adding a test case here: https://github.com/facebook/relay/blob/f4c37a333575341b78979106084068cb6b27e1cf/packages/relay-runtime/store/__tests__/resolvers/ResolverGC-test.js so that we don't regress this again? |
9ec3aee to
44f3de9
Compare
|
@captbaritone added the test. manually verified that the test fails before the fix and passes after the fix |
|
bummed this didn't make it into 18.2 |
|
Sorry for the poor communication. We are in a code freeze currently so not landing any high risk changes that are not necessary, so I wanted to wait to land this one. I'm out next week, but can try to get this landed when I return. |
|
@captbaritone has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
|
@captbaritone merged this pull request in 1e60e47. |
fixes #4830 by ensuring that there is only a single path for evicting stuff, ensuring that
maybeResolverSubscriptiongets called correctly