Skip to content

Optimize bandwidth exponentiation helper and add coverage#1188

Merged
oferchen merged 1 commit into
masterfrom
implement-missing-components-for-oc-rsync
Oct 26, 2025
Merged

Optimize bandwidth exponentiation helper and add coverage#1188
oferchen merged 1 commit into
masterfrom
implement-missing-components-for-oc-rsync

Conversation

@oferchen
Copy link
Copy Markdown
Owner

Summary

  • replace the token bucket exponent helper with an exponentiation-by-squaring implementation to keep overflow detection efficient
  • extend the bandwidth parser tests to cover the helper directly, including overflow reporting

Testing

  • cargo test -p rsync-bandwidth

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

@oferchen oferchen merged commit dc68265 into master Oct 26, 2025
@oferchen oferchen deleted the implement-missing-components-for-oc-rsync branch October 26, 2025 23:51
oferchen added a commit that referenced this pull request May 5, 2026
…1637) (#3721)

Audit current pool defaults (COPY_BUFFER_SIZE = 128 KiB,
max_buffers = available_parallelism()), document the workloads they were
tuned for, and call out where the defaults underprovision (1M-file
parallel bursts) and overprovision (sequential single-threaded
transfers, long-lived daemon mode). Cover the adaptive grow/shrink
policy from #1638-#1641, the OC_RSYNC_BUFFER_POOL_SIZE and
OC_RSYNC_BUFFER_POOL_STATS env vars from #1643, and the memory cap
interaction from #1188. Close with sizing rules of thumb for operators
and open questions for the pending #1642 benchmark.
oferchen added a commit that referenced this pull request May 18, 2026
…1637) (#3721)

Audit current pool defaults (COPY_BUFFER_SIZE = 128 KiB,
max_buffers = available_parallelism()), document the workloads they were
tuned for, and call out where the defaults underprovision (1M-file
parallel bursts) and overprovision (sequential single-threaded
transfers, long-lived daemon mode). Cover the adaptive grow/shrink
policy from #1638-#1641, the OC_RSYNC_BUFFER_POOL_SIZE and
OC_RSYNC_BUFFER_POOL_STATS env vars from #1643, and the memory cap
interaction from #1188. Close with sizing rules of thumb for operators
and open questions for the pending #1642 benchmark.
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