Skip to content

feat: add detailed error handling for job deserialization in jobs.rs#1846

Closed
techvoyagerX wants to merge 1 commit intorust-lang:masterfrom
techvoyagerX:improve-job-deserialization-error-handling
Closed

feat: add detailed error handling for job deserialization in jobs.rs#1846
techvoyagerX wants to merge 1 commit intorust-lang:masterfrom
techvoyagerX:improve-job-deserialization-error-handling

Conversation

@techvoyagerX
Copy link

This PR introduces enhanced error handling in the deserialize_job function within jobs.rs. The update provides field-specific error messages, improving traceability and making it easier to identify the source of deserialization failures. Each field (id, name, scheduled_at, metadata, executed_at, and error_message) now has its own error context, which helps with debugging and improves overall reliability.

@ehuss
Copy link
Contributor

ehuss commented Sep 18, 2024

Thanks for the PR! These don't look like functions that should fail, and we would prefer to not add this.

@ehuss ehuss closed this Sep 18, 2024
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