temporalio / sdk-core

Core Temporal SDK that can be used as a base for language specific Temporal SDKs
MIT License
278 stars 77 forks source link

[Bug] WFT Spam loop when failing WFT with update #615

Closed Sushisource closed 1 year ago

Sushisource commented 1 year ago

While working on update implementation in python, it looks like Core fails to not respond with WFT failure when processing an update job, thus creating a spam loop. This ought to not happen because we should notice the failures are the same and avoid replying like we normally should. Might be a matter of expecting too exact of a match with the failures.

Sushisource commented 1 year ago

Forgot to resolve when https://github.com/temporalio/sdk-core/pull/616 was closed