fix(global-db): defer LCM index convergence after daemon admission - #769
Conversation
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c54312f3c7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
707d596 to
b88d1c1
Compare
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
BleedingDev
changed the title
|
@codex review |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Move missing LCM status indexes into lifecycle-owned registered-schema convergence so daemon admission and ordinary reads do not wait on historical full-table builds. Keep each index in an authority-revalidated durable batch, while preserving synchronous fresh installs and short-lived attaches. Add deterministic readiness, eventual-convergence, data-preservation, and migration-marker coverage.
b88d1c1 to
4037377
Compare
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
1 participant
Summary
Why
Current-version historical stores may need full-table status-index builds. Performing those builds inside daemon admission can delay startup and ordinary retrieval even though the daemon already owns a tracked post-admission convergence lane.
Pinned rebase/test snapshot from #707:
5cea70e29d7f0a56f87d97e4cd7f28085494c2d2.Verification (macOS)
tracedecay-global-db: 343/343tracedecay-lcm: 163 passed, 1 intentional performance test ignoredrustfmt --checkandgit diff --check: clean