Skip to content

Harden rmap ordering and retry ticker advances#69

Merged
raphael merged 3 commits intomainfrom
rmap-ordering-and-ticker-retries
Mar 23, 2026
Merged

Harden rmap ordering and retry ticker advances#69
raphael merged 3 commits intomainfrom
rmap-ordering-and-ticker-retries

Conversation

@raphael
Copy link
Copy Markdown
Member

@raphael raphael commented Mar 23, 2026

Summary

  • order replicated-map writes by committed revision so destroy/reconnect recovery and SetAndWait acknowledgements stay consistent under concurrent updates
  • route stream sink cleanup through the rmap destroy protocol and preserve destroy tombstones as intentional protocol state in tests
  • retry distributed ticker advances after transient map write failures and add the repo AGENTS.md guidance in-tree

Test plan

  • Run go test ./rmap ./streaming ./pool in a Redis-backed environment
  • Verify rmap.Destroy() and reconnect keep revision ordering intact across replica resync
  • Verify a transient ticker map write failure rearms the local timer and resumes ticking without unrelated map traffic

Notes

  • I reviewed the existing local commit 3a2cc7f before opening this PR and did not find a blocker that required changes before shipping.
  • Local go test ./pool currently fails during Redis test setup with EOF, so I did not claim a clean package test run in this environment.

raphael added 3 commits March 13, 2026 08:42
Track replicated-map writes by committed revision so destroy/reconnect recovery and SetAndWait acknowledgements stay ordered under concurrent updates. Align stream sink cleanup with the same destroy protocol and treat destroy tombstones as intentional protocol state in tests.
Rearm the local timer after transient map write failures so distributed tickers recover without waiting for unrelated map traffic. Add regression coverage for the failed advance path.
Record the repository-specific architecture and coding rules in-tree so future changes follow Pulse's contracts and review expectations consistently.
@raphael raphael merged commit 90ef382 into main Mar 23, 2026
5 checks passed
@raphael raphael deleted the rmap-ordering-and-ticker-retries branch March 23, 2026 20:13
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.

1 participant