Skip to content

[PLUGIN-1851] Show correct DB name in error string#545

Merged
psainics merged 1 commit intodata-integrations:developfrom
cloudsufi:patch/plugin-1851
Jan 30, 2025
Merged

[PLUGIN-1851] Show correct DB name in error string#545
psainics merged 1 commit intodata-integrations:developfrom
cloudsufi:patch/plugin-1851

Conversation

@psainics
Copy link
Contributor

@psainics psainics commented Jan 27, 2025

Show correct DB name in error string

Jira : PLUGIN-1851

Description

Show correct DB name in error string

Code change

  • Modified CloudSQLMySQLAction.java
  • Modified CloudSQLMySQLSink.java
  • Modified CloudSQLMySQLSource.java
  • Modified CloudSQLPostgreSQLAction.java
  • Modified CloudSQLPostgreSQLSink.java
  • Modified CloudSQLPostgreSQLSource.java
  • Modified CloudSQLUtil.java

Tests

Cloud MySQL source -public

Screenshot 2025-01-27 at 10 59 16 AM

Cloud MySQL source -private

Screenshot 2025-01-27 at 10 59 31 AM

Cloud MySQL sink -public

Screenshot 2025-01-27 at 11 00 02 AM

Cloud MySQL sink -private

Screenshot 2025-01-27 at 11 00 17 AM

Cloud PostgreSQL source -public

Screenshot 2025-01-27 at 11 00 36 AM

Cloud PostgreSQL source -private

Screenshot 2025-01-27 at 11 00 51 AM

Cloud PostgreSQL sink -public

Screenshot 2025-01-27 at 11 32 43 AM

Cloud PostgreSQL sink -private

Screenshot 2025-01-27 at 11 36 10 AM

Cloud MySQL action -public

Screenshot 2025-01-27 at 3 08 13 PM

Cloud MySQL action -private

Screenshot 2025-01-27 at 3 09 49 PM

Cloud PostgreSQL action -public

Screenshot 2025-01-27 at 3 12 03 PM

Cloud PostgreSQL action -private

Screenshot 2025-01-27 at 3 13 20 PM

@psainics psainics marked this pull request as ready for review January 30, 2025 04:19
@psainics psainics merged commit 60d9e24 into data-integrations:develop Jan 30, 2025
8 of 10 checks passed
@psainics psainics deleted the patch/plugin-1851 branch January 30, 2025 05:06
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