Deferred / conditional. Factory-ise the read messages of AppsActor only if benchmarks justify it.
Context
AppsActor (src/db/apps_actor.rs:15) mixes reads (List / Get / ListAvailable, :80) and admin writes (Upsert / Remove, :63) that also poke the download actor. Apps reads are low-volume and cheap compared to item detail, so the win is marginal.
Options if pursued
- Split into a read factory (
List/Get/ListAvailable) + a single writer actor for Upsert/Remove (kept coordinated with the download actor).
- Leave as-is.
Recommendation
Defer until the ItemActor benchmark data shows apps reads are actually a bottleneck.
Depends on
Parent: #51
Deferred / conditional. Factory-ise the read messages of
AppsActoronly if benchmarks justify it.Context
AppsActor(src/db/apps_actor.rs:15) mixes reads (List/Get/ListAvailable,:80) and admin writes (Upsert/Remove,:63) that also poke the download actor. Apps reads are low-volume and cheap compared to item detail, so the win is marginal.Options if pursued
List/Get/ListAvailable) + a single writer actor forUpsert/Remove(kept coordinated with the download actor).Recommendation
Defer until the
ItemActorbenchmark data shows apps reads are actually a bottleneck.Depends on
Parent: #51