Skip to content

[PLUGIN-1829] Add CloudSQLMySQLErrorDetailsProvider#526

Merged
psainics merged 1 commit intodata-integrations:developfrom
cloudsufi:fem/cloudsql-mysql
Dec 17, 2024
Merged

[PLUGIN-1829] Add CloudSQLMySQLErrorDetailsProvider#526
psainics merged 1 commit intodata-integrations:developfrom
cloudsufi:fem/cloudsql-mysql

Conversation

@psainics
Copy link
Contributor

@psainics psainics commented Dec 12, 2024

ErrorDetailsProvider - CloudSQLMySQL [Source|Sink] plugin

Jira : PLUGIN-1829

Description

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

Code change

  • Added CloudSQLMySQLErrorDetailsProvider.java
  • Modified CloudSQLMySQLSink.java
  • Modified CloudSQLMySQLSource.java

E2E Changes

  • Adding Open and capture logs step when pipeline is expected to fail.

@psainics psainics added the build label Dec 12, 2024
@psainics psainics self-assigned this Dec 12, 2024
@psainics psainics marked this pull request as ready for review December 13, 2024 05:23
/**
* A custom ErrorDetailsProvider for CloudSQL MySQL plugins.
*/
public class CloudSQLMySQLErrorDetailsProvider extends DBErrorDetailsProvider {
Copy link
Contributor

Choose a reason for hiding this comment

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

why can't it just extend MySQLErrorDetailsProvider?

Copy link
Contributor Author

@psainics psainics Dec 13, 2024

Choose a reason for hiding this comment

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

Good, this will save duplicate code. 👍
Updated !


@Override
protected String getExternalDocumentationLink() {
return "https://dev.mysql.com/doc/mysql-errors/9.0/en/ ,https://cloud.google.com/sql/docs/mysql/error-messages";
Copy link
Contributor

Choose a reason for hiding this comment

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

Lets only use one link https://cloud.google.com/sql/docs/mysql/error-messages, the mysql one is mentioned in the cloud sql one anyways.

@psainics psainics force-pushed the fem/cloudsql-mysql branch 2 times, most recently from 2705f80 to 29a5f3d Compare December 17, 2024 09:14
@psainics psainics merged commit 2751b6d into data-integrations:develop Dec 17, 2024
9 checks passed
@psainics psainics deleted the fem/cloudsql-mysql branch December 17, 2024 12:43
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