Skip to content

feat(sync): add progress block and detailed logs - #25

Closed
hxu wants to merge 1 commit into
vicentereig:mainfrom
hxu:hxu/sync-progress-logging
Closed

hxu wants to merge 1 commit into
vicentereig:mainfrom
hxu:hxu/sync-progress-logging

Conversation

@hxu

@hxu hxu commented Jul 22, 2026

Copy link
Copy Markdown

Summary

  • Add worker progress display for sync producer and consumer tasks.
  • Add detailed sync request/logging plumbing and task timeout reporting.
  • Improve sync status output with active task details, elapsed time, attempts, and last error context.

Tests

  • cargo test

Notes

  • hxu/fix-sync-resume-counts is intentionally left stacked until this PR merges.

@vicentereig

Copy link
Copy Markdown
Owner

I took a focused slice of this work into a replacement PR: #26.

The simple outcome we want is better visibility while a long sync is running. The user should be able to see:

  • the overall totals
  • what each fetch worker (P) is doing
  • what each write worker (C) is doing
  • the task, state, and elapsed time

#26 contains only that UI/state work. It is one commit and does not change queue claiming, retry behavior, stale-claim ownership, logging, timeout policy, or resume semantics.

What should stay out of this PR:

Those are useful ideas, but they are separate decisions and make the change harder to review. The queue correctness work from #19, #20, and #23 is already on main and should remain untouched.

My recommendation is to use #26 as the focused replacement and close this PR as superseded once #26 is reviewed.

@vicentereig

Copy link
Copy Markdown
Owner

Superseded by the focused worker-progress implementation in #26, now merged.

@vicentereig vicentereig closed this Aug 8, 2026
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.

2 participants