kvcache eviction only addresses transient cache and limits should only hit that, but the bar chart needs to list sessions as a bucket #27

New Issue

hugo · 2026-07-26T07:39:36Z

hugo commented

2026-07-26 07:39:36 +00:00

title says it: kvcache budget should only target transient kvcache, not sessions, since sessions are only evicted on session delete. but the sum of sessions should also show in the bar chart and be report as part of the stats, so the user can see how much the non-auto-evicted session cache takes up in relation to transient kvcache.

hugo added the enhancement label 2026-07-26 07:39:36 +00:00

hugo commented

2026-07-26 08:00:19 +00:00

Done in f2133d5.

The budget side already held: KvStore is only ever opened on kv-cache/http, so eviction never sees session checkpoints — they live in the parent directory and go with their session.

The chart was the actual bug: the age buckets counted session files too, mixing non-evictable bytes into the eviction order. Now age_bytes covers the transient store only, and the bar plus legend carry Sessions as one further segment in its own colour, with percentages still against the combined total — so the session share reads directly against the transient buckets.

Done in f2133d5. The budget side already held: KvStore is only ever opened on kv-cache/http, so eviction never sees session checkpoints — they live in the parent directory and go with their session. The chart was the actual bug: the age buckets counted session files too, mixing non-evictable bytes into the eviction order. Now age_bytes covers the transient store only, and the bar plus legend carry Sessions as one further segment in its own colour, with percentages still against the combined total — so the session share reads directly against the transient buckets.

hugo closed this issue

2026-07-26 08:00:24 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: hugo/DS4Server#27