Skip to content

[stable34] fix(files_reminders): keep folder preload for folders over 512 entries - #65475

Open
backportbot[bot] wants to merge 1 commit into
stable34from
backport/64706/stable34
Open

backportbot[bot] wants to merge 1 commit into
stable34from
backport/64706/stable34

Conversation

@backportbot

@backportbot backportbot Bot commented Oct 13, 2026

Copy link
Copy Markdown

Backport of PR #64706

@AndyScherzinger
AndyScherzinger force-pushed the backport/64706/stable34 branch from d5a05e4 to c74477f Compare November 4, 2026 22:02
@AndyScherzinger
AndyScherzinger requested a review from a team as a code owner November 4, 2026 22:02
@AndyScherzinger
AndyScherzinger requested review from GhassenKefi and pabzm and removed request for a team November 4, 2026 22:02
ReminderService::cacheFolder() stored one entry per child in the in-memory
cache, which is capped at 512 entries. Listing a folder with 512 or more
children evicted entries before the DAV plugin read them, so each evicted
file fell back to its own query.

Keep the folder preload in a separate per-request map and drop the entry
whenever a reminder is written, so later changes in the same request win.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: John Molakvoæ <14975046+skjnldsv@users.noreply.github.com>
@skjnldsv
skjnldsv force-pushed the backport/64706/stable34 branch from c74477f to bc8cc8c Compare November 5, 2026 14:48
This was referenced Nov 10, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants