Skip to content

Commit ec9961d

Browse files
cmd/codeaf: the model-warm rows name a catalog warm that closeAll now joins
The change note takes the PR number (1264 named a PR that has already landed). With the catalog warm owned, cancelled and joined through Models.Close, the chatv3/models and engine/models rows in the guard.Go inventory law no longer carry an open writer one package down: each names the waiter joined since #1179 and the internal/catalog warm that v3Process.closeAll now cancels and joins. chatv3/sweep-home is the one known-open row left.
1 parent d18ec8d commit ec9961d

2 files changed

Lines changed: 5 additions & 5 deletions

File tree

‎cmd/codeaf/chatv3_guard_class_test.go‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@ import (
1414
// TestV3ProcessGuardGoClass is the inventory law for goroutines started by a
1515
// v3 process or launch. A long-lived writer must identify both its stop and the
1616
// closeAll join. chatv3/models and engine/models name the process waiter, joined
17-
// since #1179; the internal/catalog catalog/warm goroutine each one starts is a
18-
// separate open writer one package down, owned by the catalog-warm-close cell.
17+
// since #1179, and the internal/catalog warm each one starts, which closeAll now
18+
// cancels and joins through Models.Close.
1919
// The remaining known-open row (chatv3/sweep-home) names the later cell that owns it;
2020
// deleting a row is part of landing that cell.
2121
func TestV3ProcessGuardGoClass(t *testing.T) {
@@ -26,8 +26,8 @@ func TestV3ProcessGuardGoClass(t *testing.T) {
2626
"chatv3/standing": {kind: "joined writer", stop: "v3Process.standingStop", join: "v3Process.closeAll calls stopStandingTicks"},
2727
"chatv3/standing-stop": {kind: "joined helper", stop: "standingStop is closed", join: "chatv3/standing closes standingDone after this helper returns"},
2828
"pool/judge-sweep": {kind: "joined writer", stop: "pool errand context", join: "v3Process.closeAll calls stopPoolErrands"},
29-
"chatv3/models": {kind: "joined waiter, downstream catalog/warm still open", stop: "warmModels waiter on the pool errand context", join: "v3Process.closeAll calls stopPoolErrands, joining the waiter since #1179; the internal/catalog catalog/warm it starts is not yet joined, owned by the catalog-warm-close cell"},
30-
"engine/models": {kind: "joined waiter, downstream catalog/warm still open", stop: "warmModels waiter on the pool errand context", join: "v3Process.closeAll calls stopPoolErrands, joining the waiter since #1179; the internal/catalog catalog/warm it starts is not yet joined, owned by the catalog-warm-close cell"},
29+
"chatv3/models": {kind: "joined writer", stop: "warmModels waiter on the pool errand context, and the catalog's own warm context", join: "v3Process.closeAll calls stopPoolErrands for the waiter (since #1179) and Models.Close, which cancels and joins the internal/catalog warm"},
30+
"engine/models": {kind: "joined writer", stop: "warmModels waiter on the pool errand context, and the catalog's own warm context", join: "v3Process.closeAll calls stopPoolErrands for the waiter (since #1179) and Models.Close, which cancels and joins the internal/catalog warm"},
3131
"chatv3/sweep-home": {kind: "KNOWN-OPEN writer", owner: "later home-sweep ownership cell (class target three)"},
3232
"chatv3/background": {kind: "one-shot", stop: "repairBackgroundChecks returns after one bounded Drift/Install pass"},
3333
"chatv3/once-questions": {kind: "one-shot", stop: "agent.Close closes the WatchQuestions channel consumed by the range"},

docs/changes/unreleased/1264-catalog-lazy-warm-close.md renamed to docs/changes/unreleased/1274-catalog-lazy-warm-close.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
---
22
kind: fixed
33
title: closing a process cancels and joins lazy catalog warming before a late cache write
4-
pr: 1264
4+
pr: 1274
55
surface: [engine]
66
invalidates:
77
- "Lazy catalog warming could outlive process close and write its cache afterward, including beneath a home selected by a later launch. Close now cancels and joins the warm, and a fetch that succeeds after cancellation does not write the cache."

0 commit comments

Comments
 (0)