Skip to content

Do not return no poller errors for nexus tasks#9307

Merged
bergundy merged 1 commit intotemporalio:mainfrom
bergundy:fix-no-poller-error-for-nexus
Feb 13, 2026
Merged

Do not return no poller errors for nexus tasks#9307
bergundy merged 1 commit intotemporalio:mainfrom
bergundy:fix-no-poller-error-for-nexus

Conversation

@bergundy
Copy link
Copy Markdown
Member

@bergundy bergundy commented Feb 12, 2026

Why?

It's a failed precondition error that isn't translated properly to Nexus error semantics and becomes a non retryable error.

This is only relevant for the priority matcher, the "old" matcher already avoided this.

@bergundy bergundy requested review from a team as code owners February 12, 2026 21:55
Copy link
Copy Markdown
Contributor

@rkannan82 rkannan82 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would need this for sending activity cancelation via nexus 😄

@bergundy bergundy enabled auto-merge (squash) February 12, 2026 22:34
@bergundy bergundy merged commit 846b118 into temporalio:main Feb 13, 2026
116 of 118 checks passed
@bergundy bergundy deleted the fix-no-poller-error-for-nexus branch February 13, 2026 04:51
lilydoar pushed a commit that referenced this pull request Feb 17, 2026
## Why?

It's a failed precondition error that isn't translated properly to Nexus
error semantics and becomes a non retryable error.

This is only relevant for the priority matcher, the "old" matcher
already avoided this.
@temporal-cicd temporal-cicd Bot mentioned this pull request Feb 17, 2026
temporal-cicd Bot pushed a commit that referenced this pull request Feb 17, 2026
## Why?

It's a failed precondition error that isn't translated properly to Nexus
error semantics and becomes a non retryable error.

This is only relevant for the priority matcher, the "old" matcher
already avoided this.
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