Skip to content

Added application error category #277

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jun 3, 2025

Conversation

maciejdudko
Copy link
Contributor

What was changed

Added category field and associated enum to ApplicationError.

Why?

Feature request: temporalio/features#614

Checklist

  1. Closes [Feature Request] Apply application failure logging and metrics behaviour according to ApplicationErrorCategory #242

  2. How was this tested: added benign failure test case to worker_activity_test.rb#test_failure.

@maciejdudko maciejdudko requested a review from a team as a code owner June 2, 2025 20:05
Copy link
Member

@cretz cretz left a comment

Choose a reason for hiding this comment

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

Looks great

@maciejdudko maciejdudko merged commit 143e257 into temporalio:main Jun 3, 2025
7 checks passed
@maciejdudko maciejdudko deleted the application-error-category branch June 3, 2025 13:15
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.

[Feature Request] Apply application failure logging and metrics behaviour according to ApplicationErrorCategory
2 participants