Skip to content

CS-1992: Skip claim race fix - #123

Open
alexlivekit wants to merge 1 commit into
mainfrom
alex/skip-claim
Open

CS-1992: Skip claim race fix#123
alexlivekit wants to merge 1 commit into
mainfrom
alex/skip-claim

Conversation

@alexlivekit

Copy link
Copy Markdown
Contributor

No description provided.

erikhortsch added a commit that referenced this pull request Aug 21, 2026
CS-1992: a handler that errors on receipt publishes its response right behind
the announcement, on a different channel, and nothing orders their delivery.
When the response won the race, selection stashed it as a fallback and the
announcement then sent the caller off to wait on a channel already drained,
turning an instant error into a request timeout.

The stash exists for broadcast, where an early error is one server rejecting a
request it could not read and another may yet bid. On a queue rpc the responder
is the only server that received the request, so nothing else can ever arrive:
return the response the moment it appears. This also stops a queue caller from
waiting out the selection timeout to surface a malformed-request rejection, the
case with no announcement at all.

Reported in #123, which surfaced the stashed error once the announcement
arrived; resolving it at the response site keeps selection from consuming the
answer in the first place.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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