Commit 961069b
authored
Fault Injection: execute operation while still returning an error to the caller (temporalio#5869)
## What changed?
<!-- Describe what has changed in this PR -->
Added new fault injection error `ExecuteAndTimeout`.
While working on this, I also figured out that fault injection code
required refactoring and renames. I will do it in separate PR.
## Why?
<!-- Tell your future self why have you made these changes -->
`ExecuteAndTimeout` is a special error which emulates case, when client
got a `Timeout` error, but operation actually reached persistence and
was executed successfully.
## How did you test it?
<!-- How have you verified this change? Tested locally? Added a unit
test? Checked in staging env? -->
Run locally.
## Potential risks
<!-- Assuming the worst case, what can be broken when deploying this
change to production? -->
No risks.
## Documentation
<!-- Have you made sure this change doesn't falsify anything currently
stated in `docs/`? If significant
new behavior is added, have you described that in `docs/`? -->
No.
## Is hotfix candidate?
<!-- Is this PR a hotfix candidate or does it require a notification to
be sent to the broader community? (Yes/No) -->
No.1 parent 513817e commit 961069b
4 files changed
Lines changed: 322 additions & 339 deletions
File tree
- common/persistence/client
- config
0 commit comments