Skip to content

fix(python): bump lease_rotate_interval_ms default 50 -> 250 to match Rust/docs#322

Merged
hardbyte merged 1 commit into
issue-295-v06-sequence-cursorsfrom
copilot/review-and-verify-tla-models
Jun 5, 2026
Merged

fix(python): bump lease_rotate_interval_ms default 50 -> 250 to match Rust/docs#322
hardbyte merged 1 commit into
issue-295-v06-sequence-cursorsfrom
copilot/review-and-verify-tla-models

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jun 5, 2026

Summary

The Python client still defaulted queue_storage_lease_rotate_interval_ms to 50, while the Rust default and docs moved to 250ms in #169 (and carried through #321). This drift meant Python users got a different lease-rotation cadence than the documented/Rust default unless they passed the argument explicitly.

This brings the Python bindings in line:

  • awa-python/src/client.rs — pyo3 start signature default 50250
  • awa-python/python/awa/client.pyAsyncClient keyword default 50250
  • awa-python/python/awa/_awa.pyi — type stub default 50250

Context

Ideally this would land on the open PR #321, but that PR's branch (issue-295-v06-sequence-cursors) isn't writable from this environment, so it's surfaced here as a standalone follow-up. It is a one-commit change stacked on top of #321's head.

Validation

Copilot AI requested a review from hardbyte June 5, 2026 22:52
@hardbyte hardbyte changed the base branch from main to issue-295-v06-sequence-cursors June 5, 2026 22:54
@hardbyte hardbyte marked this pull request as ready for review June 5, 2026 22:54
@hardbyte hardbyte merged commit 8820b31 into issue-295-v06-sequence-cursors Jun 5, 2026
2 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.

2 participants