Skip to content

Update Error message to include sql error codes#1929

Merged
itsankit-google merged 1 commit intocdapio:developfrom
cloudsufi:patch/error-message-db-error-details-provider
Jan 29, 2025
Merged

Update Error message to include sql error codes#1929
itsankit-google merged 1 commit intocdapio:developfrom
cloudsufi:patch/error-message-db-error-details-provider

Conversation

@psainics
Copy link
Contributor

@psainics psainics commented Jan 29, 2025

Update Error message to include sql error codes

data-integrations/database-plugins#543

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

Test [ Database Plugin [ mysql] ]

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 force-pushed the patch/error-message-db-error-details-provider branch from 95073ec to c2b2cee Compare January 29, 2025 06:26
@psainics psainics marked this pull request as ready for review January 29, 2025 06:57
@itsankit-google itsankit-google merged commit 780dbde into cdapio:develop Jan 29, 2025
5 checks passed
@psainics psainics deleted the patch/error-message-db-error-details-provider branch January 29, 2025 07:25
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