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

[PLUGIN-1824] ErrorDetailsProvider - MySql Source/Sink plugin #523

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

psainics
Copy link
Contributor

@psainics psainics commented Nov 20, 2024

ErrorDetailsProvider - MySql [Source|Sink] plugin

Jira : PLUGIN-1824

Description

Implement Program Failure Exception Handling in MySql Source/Sink plugin to catch known errors

Code change

  • Added DBErrorDetailsProvider.java
  • Added MysqlErrorDetailsProvider.java
  • Modified DBRecord.java
  • Modified AbstractDBSink.java
  • Modified AbstractDBSource.java
  • Modified MysqlConstants.java
  • Modified MysqlSink.java
  • Modified MysqlSource.java
  • Modified MysqlUtil.java

@psainics psainics changed the title Add MysqlErrorDetailsProvider [PLUGIN-0000] ErrorDetailsProvider - MySql Source/Sink plugin Nov 20, 2024
@psainics psainics changed the title [PLUGIN-0000] ErrorDetailsProvider - MySql Source/Sink plugin [PLUGIN-1824] ErrorDetailsProvider - MySql Source/Sink plugin Nov 20, 2024
@psainics psainics marked this pull request as ready for review November 20, 2024 11:57
@psainics psainics self-assigned this Nov 20, 2024
@psainics psainics marked this pull request as draft November 20, 2024 11:58
@psainics psainics added the build label Nov 20, 2024
@@ -114,7 +119,7 @@ public void configurePipeline(PipelineConfigurer pipelineConfigurer) {

if (sourceConfig.canConnect()) {
try {
stageConfigurer.setOutputSchema(getSchema(driverClass));
stageConfigurer.setOutputSchema(getSchema(driverClass, collector));
Copy link
Member

Choose a reason for hiding this comment

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

we are already adding the exception in failure collector below by catching it, why is this extra step needed?

Copy link
Member

Choose a reason for hiding this comment

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

also can you please add sql state and error code in the message on line 127?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added SQL error details.

@psainics psainics marked this pull request as ready for review November 22, 2024 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants