Skip to content
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

[🍒] Update Error message to include sql error codes #1930

Conversation

psainics
Copy link
Contributor

@psainics psainics commented Jan 29, 2025

🍒 [cherrypick]

Commits :

PR:


Update Error message to include sql error codes

data-integrations/database-plugins#543

data-integrations/database-plugins#543 (comment)

Test [ Database Plugin [ mysql] ] [ release/2.13 ]

image

[
  {
    "stageName": "Database2",
    "errorCategory": "Plugin-'DB Integrity Constraint Violation'-'Database2'",
    "errorReason": "SQL Exception occurred: [Message='Duplicate entry '5' for key 'PRIMARY'', SQLState='23000', ErrorCode='1062']. For more details, see https://en.wikipedia.org/wiki/SQLSTATE",
    "errorMessage": "Error occurred in the phase: 'Writing' with sqlState: '23000', errorCode: '1062', errorMessage: SQL Exception occurred: [Message='Duplicate entry '5' for key 'PRIMARY'', SQLState='23000', ErrorCode='1062'].",
    "errorType": "USER",
    "dependency": "true",
    "errorCodeType": "SQLSTATE",
    "errorCode": "23000",
    "supportedDocumentationUrl": "https://en.wikipedia.org/wiki/SQLSTATE"
  }
]

@psainics psainics added the build Trigger unit test build label Jan 29, 2025
@psainics psainics self-assigned this Jan 29, 2025
@psainics psainics marked this pull request as ready for review January 29, 2025 07:37
@itsankit-google itsankit-google merged commit 0a2c026 into cdapio:release/2.13 Jan 29, 2025
7 checks passed
@psainics psainics deleted the cherry-pick/c2b2cee78401a393042861336eeb9e3e87c505e2 branch January 29, 2025 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Trigger unit test build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants