Skip to content

fix(p2p-11): harden reader and listener lifecycle (#671)#7

Open
jlon wants to merge 1 commit into
p2p-10-correctness-hardeningfrom
p2p-11-reader-listener-lifecycle-hardening
Open

fix(p2p-11): harden reader and listener lifecycle (#671)#7
jlon wants to merge 1 commit into
p2p-10-correctness-hardeningfrom
p2p-11-reader-listener-lifecycle-hardening

Conversation

@jlon
Copy link
Copy Markdown
Owner

@jlon jlon commented Mar 26, 2026

Summary

  • Harden reader failure handling and data-plane listener lifecycle.
  • Make this the first redirect slice that is reasonable for rollout when stacked on p2p-10.

Design

  • Keep worker fallback correct and prevent broken remote readers from re-entering the pool.
  • Isolate listener retry, shutdown, and generation safety from later performance work.

Implementation

  • Fix last-replica failure handling, remote reader cleanup, and block client pool hygiene.
  • Add listener retry/stop/start hardening and the FsContext and pool lifetime fixes needed for P2P lifecycle.

Verification

  • Verify broken remote readers are not returned to the pool.
  • Verify stop/start is clean and transient listener failures self-recover.

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@jlon jlon force-pushed the p2p-10-correctness-hardening branch from c5bbf47 to a85fe48 Compare March 26, 2026 10:26
@jlon jlon force-pushed the p2p-11-reader-listener-lifecycle-hardening branch from bd6a1a3 to 3da3eea Compare March 26, 2026 10:26
@jlon jlon force-pushed the p2p-10-correctness-hardening branch from a85fe48 to 5ab0e12 Compare March 27, 2026 09:33
@jlon jlon force-pushed the p2p-11-reader-listener-lifecycle-hardening branch from 3da3eea to a0d27ba Compare March 27, 2026 09:33
@jlon jlon force-pushed the p2p-10-correctness-hardening branch from 5ab0e12 to 62fb81b Compare March 30, 2026 00:55
@jlon jlon force-pushed the p2p-11-reader-listener-lifecycle-hardening branch from a0d27ba to 50bb594 Compare March 30, 2026 00:55
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