Skip to content

Backfill orchestrator should log outcomes and preserve tracebacks#569

Merged
jirhiker merged 2 commits intostagingfrom
fix/backfill-orchestrator-logging
Mar 2, 2026
Merged

Backfill orchestrator should log outcomes and preserve tracebacks#569
jirhiker merged 2 commits intostagingfrom
fix/backfill-orchestrator-logging

Conversation

@kbighorse
Copy link
Contributor

@kbighorse kbighorse commented Feb 28, 2026

Summary

  • Capture and log BackfillResult summary (inserted/updated/skipped_orphans/errors) instead of discarding the return value
  • Log individual errors at WARNING level for operator visibility
  • Preserve full traceback on critical failure via exc_info=True instead of f-string

Refs

Closes #563, closes #564
Part of #558

Blocks

These issues depend on code introduced by #558 and will be addressed on that feature branch.

Test plan

🤖 Generated with Claude Code

Copilot AI review requested due to automatic review settings February 28, 2026 03:19
Capture and log the BackfillResult summary (inserted/updated/skipped/errors)
instead of discarding it. Preserve full traceback on critical failure via
exc_info=True.

Closes #563, closes #564
Refs #558

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the backfill orchestrator’s logging so operators can see per-step outcomes and get full tracebacks on orchestration failures.

Changes:

  • Capture each backfill step’s return value and log inserted/updated/skipped_orphans/error counts.
  • Log individual step errors at WARNING level for visibility.
  • Preserve traceback on critical failures by using exc_info=True in the __main__ exception handler.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jirhiker jirhiker merged commit 7895e8d into staging Mar 2, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Backfill critical logs should preserve tracebacks Backfill orchestrator should log BackfillResult outcomes

3 participants