Skip to content

Update 02-error-handling.md#301

Closed
mironov-artem wants to merge 1 commit into
ash-project:mainfrom
mironov-artem:patch-2
Closed

Update 02-error-handling.md#301
mironov-artem wants to merge 1 commit into
ash-project:mainfrom
mironov-artem:patch-2

Conversation

@mironov-artem
Copy link
Copy Markdown

Contributor checklist

Leave anything that you believe does not apply unchecked.

  • I accept the AI Policy, or AI was not used in the creation of this PR.
  • Bug fixes include regression tests
  • Chores
  • Documentation changes
  • Features include unit/acceptance tests
  • Refactoring
  • Update dependencies

@jimsynz
Copy link
Copy Markdown
Contributor

jimsynz commented Apr 6, 2026

Thanks for this one too, @mironov-artem — returning {:retry, reason} from compensate/4 would have worked around the issue. We've gone with fixing the root cause in the executor instead (#306), so that backoff/4 always receives the unwrapped error regardless of whether compensate/4 returns bare :retry or {:retry, reason}. Appreciate the contribution!

@jimsynz jimsynz closed this Apr 6, 2026
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