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

Improve handling of caches having discarded content #1852

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

barijaona
Copy link
Member

You can assess the behavior changes by running sudo memory_pressure -S -l critical -s 60 in parallel of Vienna

Some of the former @synchronized occasionally led to infinite locks,
so I reviewed to just use @synchronized for cache updates.
This is more user friendly, by preserving the most commonly used sort
orders.
Ease testing of low memory situations in unified display view
Move articles in Folder's cache only after getCompleteArticles return,
so that articleController gets articles which are able to be displayed.

Also mark and verify access to articles retrieved from Folder's cache.
As a convenience after the system encountered a low memory situation.
@barijaona barijaona marked this pull request as draft November 26, 2024 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant