FROSADO / onecloudriver / 34122575811

81%

main: 81%

Build Type

Pull #154

github

Committed by FROSADO

Commit Message

fix(fs): protect in-use folders from size-limit eviction

A folder whose children were just fetched from Graph could be evicted by
the size-limit sweep before serving a single cache hit: population did not
count as an access, so a freshly populated folder scored 0 and, under
cacheMaxEntries pressure, was removed within ~1s. Every subsequent Lookup
and Readdir then re-fetched the whole listing; with a small cacheMaxEntries
this repeated forever and, combined with per-name lookups, hung file
managers for minutes (issue #152).

- Population now counts as activity (BumpChildrenAccess after SetChildren).
- Size eviction is idle-aware (sizeEvictIdleGrace=5s): a folder is only an
  eligible victim once it has been idle that long, so the folder being read
  is never evicted even when the cache is over capacity; the sweep defers
  eviction until the folder goes idle.
- Regression tests cover the reported scenario; the end-to-end test fails
  on the pre-fix code.

Pull Request Pull Request #154: fix(fs): protect in-use folders from size-limit eviction

Coverage Stats

37 of 41 new or added lines in 1 file covered. (90.24%)

4 existing lines in 2 files now uncovered.

6477 of 7976 relevant lines covered (81.21%)

3107.58 hits per line