From 005942f1413c0a750fbb5b7ccf9bff44cf9c5a3b Mon Sep 17 00:00:00 2001 From: Johnathan Corgan Date: Sun, 16 Aug 2026 07:08:40 +0000 Subject: [PATCH] Stop citing a design document that is not in the repository Three comments in the control snapshot module pointed readers at a fast-path design note that exists only in my working notes and is not a blob at any branch. Each now states the thing the document was cited for, so the comment stands on its own. The fourth citation, in the read handle's module doc, stays for now: the sentence around it is also wrong about when queries read the handle, and correcting the path alone would commit the false half. That whole block is replaced together in a later change. --- src/control/snapshot.rs | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/src/control/snapshot.rs b/src/control/snapshot.rs index 8f5f4378..c7132202 100644 --- a/src/control/snapshot.rs +++ b/src/control/snapshot.rs @@ -1,8 +1,7 @@ //! Read-side state snapshots published from the node's natural mutators so //! pure-snapshot `show_*` queries render off the rx_loop hot path. //! -//! [`StatsSnapshot`] is the R2 reference implementation of the canonical -//! snapshot pattern (see `design/fast-path-refactoring-r0-read-handle.md`): a +//! [`StatsSnapshot`] is the reference implementation of the canonical snapshot pattern: a //! read-only data bundle published via `ArcSwap` from the tick after //! `StatsHistory::tick()`. It carries //! @@ -145,8 +144,8 @@ fn empty_acl_status() -> PeerAclStatus { /// the pure-snapshot `show_tree` / `show_bloom` / `show_cache` / `show_routing` /// / `show_identity_cache` queries render. Published via `ArcSwap`. /// -/// The R0 stub (`design/fast-path-refactoring-r0-read-handle.md`) names a -/// single combined `ArcSwap` for R3. This is that cell: one +/// This is the single combined `ArcSwap` cell for the routing +/// subsystems: one /// cohesive routing view holding the four subsystems (tree / bloom / coord /// cache / identity cache) plus the F-queue summary scalars. /// @@ -405,10 +404,8 @@ pub(crate) struct IdentityRow { /// `show_connections` / `show_transports` / `show_mmp` queries render. /// Published via `ArcSwap`. /// -/// The R0 stub (`design/fast-path-refactoring-r0-read-handle.md`) pre-scopes -/// R4 as `entities — ArcSwap: peers / sessions / links / -/// connections / transports, published per-entity with `Vec>` -/// structural sharing`. This is that cell. +/// This is the `entities` cell: peers / sessions / links / connections / +/// transports, published per-entity with `Vec>` structural sharing. /// /// **Structural sharing (the umbrella mandate).** Every entity table is a /// `Vec>`, so a republish in which only one row changed re-allocates