Skip to content

core: expose NonZeroU8 protocol accessor#1543

Merged
oferchen merged 1 commit into
masterfrom
prepare-production-release-and-cleanup
Oct 29, 2025
Merged

core: expose NonZeroU8 protocol accessor#1543
oferchen merged 1 commit into
masterfrom
prepare-production-release-and-cleanup

Conversation

@oferchen
Copy link
Copy Markdown
Owner

Summary

  • expose a NonZeroU8 helper for the workspace protocol version so callers can rely on the non-zero invariant
  • extend the workspace metadata test to cover the new accessor

Testing

  • cargo test -p rsync-core workspace::tests::parse_protocol_matches_env

https://chatgpt.com/codex/tasks/task_e_69023b94e4a0832392fd727df9d0149e

@oferchen oferchen merged commit eb8ccdb into master Oct 29, 2025
@oferchen oferchen deleted the prepare-production-release-and-cleanup branch October 29, 2025 16:22
oferchen added a commit that referenced this pull request May 7, 2026
Concise design for a receiver-side multi-file delta-apply pipeline
that overlaps per-file apply across in-flight files while preserving
strict wire-NDX order at acknowledgement and disk-commit boundaries.
Composes the existing ReceiverDeltaPipeline trait (#1543), bounded
work queue, and BoundedReorderBuffer (#1407). Threshold gate at
PARALLEL_STAT_THRESHOLD=64 (#1547). Risks documented: head-of-line
stall (#1883) and spill-to-tempfile pending (#1884).
oferchen added a commit that referenced this pull request May 18, 2026
Concise design for a receiver-side multi-file delta-apply pipeline
that overlaps per-file apply across in-flight files while preserving
strict wire-NDX order at acknowledgement and disk-commit boundaries.
Composes the existing ReceiverDeltaPipeline trait (#1543), bounded
work queue, and BoundedReorderBuffer (#1407). Threshold gate at
PARALLEL_STAT_THRESHOLD=64 (#1547). Risks documented: head-of-line
stall (#1883) and spill-to-tempfile pending (#1884).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant