Repository navigation
docs: record verified analytics owner recovery - #3252
Conversation
Reviewer's GuideDocumentationRecords the analytics-owner incident, approved recovery and replay evidence, while keeping the unresolved executor termination cause and separate sink-connection follow-up distinct.
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 25 minutes. View limit details
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3252 +/- ##
==========================================
+ Coverage 96.96% 97.40% +0.44%
==========================================
Files 1146 1146
Lines 63139 63139
Branches 19623 19623
==========================================
+ Hits 61223 61501 +278
+ Misses 1866 1592 -274
+ Partials 50 46 -4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
🤖 Review skipped: Repository rate limit exceeded. Free accounts are limited to 2 reviews per 4 hours per repository. Upgrade to a paid plan for unlimited reviews. |
2 similar comments
|
🤖 Review skipped: Repository rate limit exceeded. Free accounts are limited to 2 reviews per 4 hours per repository. Upgrade to a paid plan for unlimited reviews. |
|
🤖 Review skipped: Repository rate limit exceeded. Free accounts are limited to 2 reviews per 4 hours per repository. Upgrade to a paid plan for unlimited reviews. |
|
ⓘ Qodo reviews are paused because the subscription is no longer active. Ask your workspace admin to reactivate the subscription to resume reviews. Manage billing |
Mergestorm reviews this repository's pull requestsEvery PR gets AI code review that flags real bugs before they merge. This account's 30 free credits for this monthly window are used, so reviews are paused at The Starter plan includes 120 credits a month. |
Records DOFEK-SERVER-83, where an abandoned non-quiescent publication owner blocked analytics admission across worker replacements, and the subsequently approved production recovery.
Recovery stopped cooperating writers and verified executor/request termination before releasing only the recorded owner under its advisory lock. The pending intent and ingestion reservation were preserved for replay. The normal entrypoint then passed all 46 dbt models, recorded eight datasets with zero failures, refreshed 11 caches and released the owner normally. Original replica counts and images were restored; web returned HTTP 200 and analytics readiness reported healthy.
The original executor termination path remains unconfirmed, with deployment correlation documented. The separate sink database-connection failure still requires deployment of merged PR #3250. No runtime code, image, timeout, retry or healthcheck changed in this PR.
Validation:
git diff --checkpassed. Runtime tests were skipped locally for the Markdown-only diff; hosted checks run normally. An explicit whole-file CSpell scan reported 457 historical findings outside the new entry; that documentation file is outside the configured CI spellcheck scope.Summary by Sourcery
Document the approved, evidence-backed recovery and replay of a blocked analytics publication owner without changing runtime behavior.
Enhancements:
Documentation: