Skip to content
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

Always consider ROOT_{QUERY,MUTATION,...} IDs to be GC roots. #7333

Merged
merged 2 commits into from
Nov 16, 2020

Conversation

benjamn
Copy link
Member

@benjamn benjamn commented Nov 16, 2020

Partial solution for #7323, as we still need to decide the best way to persist retainment counts for non-ROOT_* entity objects written directly with cache.writeFragment.

@benjamn
Copy link
Member Author

benjamn commented Nov 16, 2020

@hwillson @jcreighton On second thought, I'd like to get this mitigation out ASAP in v3.2.6, and do the rest of the work (preserving writeFragment retainment) in v3.3.0.

Copy link
Contributor

@jcreighton jcreighton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Member

@hwillson hwillson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great - thanks for addressing this @benjamn!

benjamn added a commit that referenced this pull request Nov 16, 2020
@benjamn benjamn force-pushed the 7323-restore-gc-root-ids branch from 7dc33c5 to a2ab26b Compare November 16, 2020 20:05
benjamn added a commit that referenced this pull request Nov 16, 2020
@benjamn benjamn force-pushed the 7323-restore-gc-root-ids branch from a2ab26b to 28e0d6e Compare November 16, 2020 20:06
Partial solution for #7323, as we still need to decide the best way to
persist retainment counts for non-ROOT_* entity objects written directly
with cache.writeFragment.
@benjamn benjamn force-pushed the 7323-restore-gc-root-ids branch from 3d1a517 to 8fa17f5 Compare November 16, 2020 20:46
@benjamn benjamn merged commit 6e1e5ad into main Nov 16, 2020
@benjamn benjamn deleted the 7323-restore-gc-root-ids branch November 16, 2020 20:58
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants