moved db version logging from mysql connector to CDK for all JDBC sou…

[Jim Ruppert (jimruppert)]

moved db version logging from mysql connector to CDK , will handle all JDBC sources
logs only during check operation

Can this PR be safely reverted and rolled back?

• YES 💚
• NO ❌

[github-actions]

👋 Greetings, Airbyte Team Member!

Here are some helpful tips and reminders for your convenience.

Helpful Resources

• Developing Connectors Locally
• Managing Connector Secrets
• On-Demand Live Tests
• On-Demand Regression Tests
• #connector-ci-issues
• #connector-publish-updates
• #connector-build-statuses

PR Slash Commands

Airbyte Maintainers (that's you!) can execute the following slash commands on your PR:

• /format-fix - Fixes most formatting issues.
• /bump-version - Bumps connector versions.
  • You can specify a custom changelog by passing changelog. Example: /bump-version changelog="My cool update"
  • Leaving the changelog arg blank will auto-populate the changelog from the PR title.
• /run-cat-tests - Runs legacy CAT tests (Connector Acceptance Tests)
• /build-connector-images - Builds and publishes a pre-release docker image for the modified connector(s).
• /poe connector source-example lock - Run the Poe lock task on the source-example connector, committing the results back to the branch.
• /poe source example lock - Alias for /poe connector source-example lock.
• /poe source example use-cdk-branch my/branch - Pin the source-example CDK reference to the branch name specified.
• /poe source example use-cdk-latest - Update the source-example CDK dependency to the latest available version.

📝 Edit this welcome message.

[github-actions]

destination-hubspot Connector Test Results

44 tests   14 ✅  27s ⏱️
 5 suites  30 💤
 5 files     0 ❌

Results for commit 7c17b6e.

♻️ This comment has been updated with latest results.

[github-actions]

destination-azure-blob-storage Connector Test Results

340 tests   220 ✅  49m 28s ⏱️
 13 suites  120 💤
 13 files      0 ❌

Results for commit 7c17b6e.

♻️ This comment has been updated with latest results.

[github-actions]

destination-dev-null Connector Test Results

47 tests   15 ✅  40s ⏱️
 4 suites  32 💤
 4 files     0 ❌

Results for commit 7c17b6e.

♻️ This comment has been updated with latest results.

[github-actions]

destination-customer-io Connector Test Results

48 tests   18 ✅  24s ⏱️
 7 suites  30 💤
 7 files     0 ❌

Results for commit 7c17b6e.

♻️ This comment has been updated with latest results.

[github-actions]

destination-clickhouse Connector Test Results

 15 files   15 suites   3m 12s ⏱️
222 tests 181 ✅ 41 💤 0 ❌
242 runs  201 ✅ 41 💤 0 ❌

Results for commit 7c17b6e.

♻️ This comment has been updated with latest results.

[github-actions]

destination-mssql Connector Test Results

135 tests   117 ✅  7m 14s ⏱️
 12 suites   18 💤
 12 files      0 ❌

Results for commit 7c17b6e.

♻️ This comment has been updated with latest results.

[github-actions]

source-mysql Connector Test Results

  8 files    8 suites   28s ⏱️
 51 tests  51 ✅ 0 💤 0 ❌
175 runs  175 ✅ 0 💤 0 ❌

Results for commit 7c17b6e.

♻️ This comment has been updated with latest results.

[github-actions]

destination-s3 Connector Test Results

808 tests   458 ✅  3h 18m 5s ⏱️
 26 suites  349 💤
 26 files      1 ❌

For more details on these failures, see this check.

Results for commit 7c17b6e.

♻️ This comment has been updated with latest results.

[github-actions]

Deploy preview for airbyte-docs ready!

✅ Preview
https://airbyte-docs-4zw1aoye8-airbyte-growth.vercel.app

Built with commit 7c17b6e.
This pull request is being automatically deployed with vercel-action

[Jim Ruppert (jimruppert)]

This PR moves the DB version logging from the mysql source connector to the CDK, so it will log for all JDBC sources.

[Yue Li (theyueli)]

To test this CDK change, we would have to change build.gradle of source-mysql to use the current local cdk, e.g.,

airbyteBulkConnector {
    core = 'extract'
    toolkits = ['extract-jdbc', 'extract-cdc']
    cdk = 'local'
}

Then, change it back to use the newly published version (after this PR is merged)

    core = 'extract'
    toolkits = ['extract-jdbc', 'extract-cdc']
    cdk = '0.1.12'
}

Unless there is a revised flow possibly done by Edward Gao (@edgao)