AH subprocess Phase 3: bound terminal worker journal retention#444
Conversation
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 18 |
| Duplication | 0 |
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: be827b55d6
ℹ️ 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".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f0e2e6fe89
ℹ️ 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".
|
@codex review |
|
Codex Review: Didn't find any major issues. You're on a roll. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
Summary
ah_worker_journalrows.COMMITTEDrows bycreated_timeandAPPLIEDrows byresolved_time.APPLIEDresolution whilecustody_ledgerstill contains itsresolve:<uuid>idempotency marker.APPLIEDtransitions from wall time so retention does not depend on the firstIPC_GAMETIMEframe.Configuration
AH.Service.JournalTerminalRetentionSec = 2592000.AH.Service.JournalAppliedRetentionSecname as a fallback so a staged0disable value remains effective.AH.Service.JournalPruneIntervalSec = 3600; setting either effective value to0disables pruning.2026071400.SELECTgrant oncustody_ledger; no custody write access is added.Database Coordination
This server change requires character DB
22.5.3from mangoszero/database#157. Apply that migration before deploying or enabling the updated worker.Safety
RESOLVING,CANCEL_PREPARED, andINTENT_PENDINGrows are never retention targets.--selftestremains config-free; an explicit--configopts into DB-backed checks without the scheduler fixture replacing global configuration.Testing
mangosd,realmd, andah-servicein Release with MSVC.ah-service --selftestruns.mangosd -t noop,-t ahforwardreserve, and-t ahmutresult.22.5.2schema and an interrupted post-DDL/pre-version state on MariaDB 10.11.22.5.3, both composite index definitions, configuration version/key parity, and clean staged diffs.This change is