Skip to content
This repository was archived by the owner on Sep 5, 2025. It is now read-only.

add actual error from query execution to the exception message#7

Merged
ujovlado merged 3 commits into
mainfrom
odin-AJDA-1025
Aug 28, 2025
Merged

add actual error from query execution to the exception message#7
ujovlado merged 3 commits into
mainfrom
odin-AJDA-1025

Conversation

@odinuv
Copy link
Copy Markdown
Member

@odinuv odinuv commented Aug 28, 2025

@odinuv
Copy link
Copy Markdown
Member Author

odinuv commented Aug 28, 2025

Release Notes Plans
Minor improvement to error handling in query-api-php-client library. Query failures now return actual SQL error messages instead of generic status codes.

Customer Communication
No direct customer communication needed - this is a library enhancement that improves developer experience. Consuming applications will automatically benefit from more detailed error messages.

Impact Analysis
Low risk change affecting only error message content, not functionality. Developers using this library will receive more actionable error information when queries fail, improving debugging capabilities.

Change Type
Bug fix/improvement - Enhanced error handling to expose actual SQL errors instead of generic "Query job failed with status: failed" messages.

Justification
Addresses AJDA-1025 under the "Augustus Secundus" epic. Current error messages provide insufficient detail for debugging failed queries, making troubleshooting difficult for developers.

Deployment Plan
Standard library release process - tag new version, publish to package manager. No deployment coordination required as this is a PHP library dependency.

Rollback Plan
Revert to previous version via composer if issues arise. Change is backward compatible and only affects error message content, not API contracts.

Post Release
Monitor error logs to ensure new error extraction logic works correctly. Verify consuming applications receive improved error details without breaking changes.

Support Plan
Update internal documentation to reference enhanced error reporting. No additional support overhead expected - change should reduce support burden by providing clearer error context.

@odinuv odinuv requested a review from ondrajodas August 28, 2025 12:30
Copy link
Copy Markdown
Contributor

@ondrajodas ondrajodas left a comment

Choose a reason for hiding this comment

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

jo dobrý 👍

@ujovlado ujovlado merged commit 4d18c0d into main Aug 28, 2025
1 check passed
@ujovlado ujovlado deleted the odin-AJDA-1025 branch August 28, 2025 13:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants