Skip to content

🎉 MySQL Source : Expose serverTimezone debezium option via MySQL Source spec for CDC - #17815

Merged
Akash Kulkarni (akashkulk) merged 19 commits into
masterfrom
akash/cdc-tz
Oct 12, 2022
Merged

Akash Kulkarni (akashkulk) merged 19 commits into
masterfrom
akash/cdc-tz

Conversation

@akashkulk

@akashkulk Akash Kulkarni (akashkulk) commented Oct 11, 2022

Copy link
Copy Markdown
Contributor

What

Closes #6573

Currently, CDC mode does not work when when the source MySQL db is set up with a server timezone that is not recognized by the ZoneID Java library (e.g. CEST, PDT).

The fix here is to expose an server timezone option in the CDC replication methods UI and add it to the debezium serverTimezone property while reading from MySQL in CDC mode.

🚨 User Impact 🚨

User will see a new option to add server timezone, with messaging that this is an 'advanced' option - only to be used when the server timezone is not IANNA compliant.

https://www.loom.com/i/2c1d080ab76541419782d508a22ab98b

@github-actions

Copy link
Copy Markdown
Contributor

NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to run corresponding integration tests:

  • source-mysql
  • source-mysql-strict-encrypt

@akashkulk

Akash Kulkarni (akashkulk) commented Oct 11, 2022

Copy link
Copy Markdown
Contributor Author

/test connector=connectors/source-mysql-strict-encrypt

🕑 connectors/source-mysql-strict-encrypt https://github.com/airbytehq/airbyte/actions/runs/3223298892
❌ connectors/source-mysql-strict-encrypt https://github.com/airbytehq/airbyte/actions/runs/3223298892
🐛 https://gradle.com/s/gfrkyuq7cl2oa

Build Failed

Test summary info:

Could not find result summary

@akashkulk

Akash Kulkarni (akashkulk) commented Oct 11, 2022

Copy link
Copy Markdown
Contributor Author

/test connector=connectors/source-mysql

🕑 connectors/source-mysql https://github.com/airbytehq/airbyte/actions/runs/3223299137
❌ connectors/source-mysql https://github.com/airbytehq/airbyte/actions/runs/3223299137
🐛 https://gradle.com/s/23ht6kdibqe26

Build Failed

Test summary info:

Could not find result summary

@akashkulk Akash Kulkarni (akashkulk) changed the title Expose serverTimezone debezium option via MySQL Source spec for CDC 🎉 Expose serverTimezone debezium option via MySQL Source spec for CDC Oct 11, 2022
@github-actions

Copy link
Copy Markdown
Contributor

NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to run corresponding integration tests:

  • source-mysql-strict-encrypt
  • source-mysql

@akashkulk

Akash Kulkarni (akashkulk) commented Oct 11, 2022

Copy link
Copy Markdown
Contributor Author

/test connector=connectors/source-mysql-strict-encrypt

🕑 connectors/source-mysql-strict-encrypt https://github.com/airbytehq/airbyte/actions/runs/3223916999
✅ connectors/source-mysql-strict-encrypt https://github.com/airbytehq/airbyte/actions/runs/3223916999
No Python unittests run

Build Passed

Test summary info:

All Passed

@akashkulk

Akash Kulkarni (akashkulk) commented Oct 11, 2022

Copy link
Copy Markdown
Contributor Author

/test connector=connectors/source-mysql

🕑 connectors/source-mysql https://github.com/airbytehq/airbyte/actions/runs/3223918088
✅ connectors/source-mysql https://github.com/airbytehq/airbyte/actions/runs/3223918088
No Python unittests run

Build Passed

Test summary info:

All Passed

@github-actions

Copy link
Copy Markdown
Contributor

NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to run corresponding integration tests:

  • source-mysql
  • source-mysql-strict-encrypt

@github-actions

Copy link
Copy Markdown
Contributor

NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to run corresponding integration tests:

  • source-mysql-strict-encrypt
  • source-mysql

@github-actions

Copy link
Copy Markdown
Contributor

NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to run corresponding integration tests:

  • source-mysql
  • source-mysql-strict-encrypt

@github-actions github-actions Bot added the area/documentation Improvements or additions to documentation label Oct 11, 2022
@github-actions

Copy link
Copy Markdown
Contributor

NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to run corresponding integration tests:

  • source-mysql-strict-encrypt
  • source-mysql

@akashkulk
Akash Kulkarni (akashkulk) marked this pull request as ready for review October 11, 2022 16:27
@akashkulk
Akash Kulkarni (akashkulk) requested a review from a team as a code owner October 11, 2022 16:27
@github-actions

Copy link
Copy Markdown
Contributor

NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to run corresponding integration tests:

  • source-mysql
  • source-mysql-strict-encrypt

@github-actions

Copy link
Copy Markdown
Contributor

NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to run corresponding integration tests:

  • source-mysql
  • source-mysql-strict-encrypt

@github-actions

Copy link
Copy Markdown
Contributor

NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to run corresponding integration tests:

  • source-mysql
  • source-mysql-strict-encrypt

@github-actions

Copy link
Copy Markdown
Contributor

NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to run corresponding integration tests:

  • source-mysql
  • source-mysql-strict-encrypt

@akashkulk

Akash Kulkarni (akashkulk) commented Oct 12, 2022

Copy link
Copy Markdown
Contributor Author

/test connector=connectors/source-mysql-strict-encrypt

🕑 connectors/source-mysql-strict-encrypt https://github.com/airbytehq/airbyte/actions/runs/3231075194
✅ connectors/source-mysql-strict-encrypt https://github.com/airbytehq/airbyte/actions/runs/3231075194
No Python unittests run

Build Passed

Test summary info:

All Passed

@github-actions

Copy link
Copy Markdown
Contributor

NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to run corresponding integration tests:

  • source-mysql
  • source-mysql-strict-encrypt

@github-actions

Copy link
Copy Markdown
Contributor

NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to run corresponding integration tests:

  • source-mysql-strict-encrypt
  • source-mysql

@akashkulk

Akash Kulkarni (akashkulk) commented Oct 12, 2022

Copy link
Copy Markdown
Contributor Author

/test connector=connectors/source-mysql

🕑 connectors/source-mysql https://github.com/airbytehq/airbyte/actions/runs/3235809223
✅ connectors/source-mysql https://github.com/airbytehq/airbyte/actions/runs/3235809223
No Python unittests run

Build Passed

Test summary info:

All Passed

@github-actions

Copy link
Copy Markdown
Contributor

NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to run corresponding integration tests:

  • source-mysql
  • source-mysql-strict-encrypt

@akashkulk

Akash Kulkarni (akashkulk) commented Oct 12, 2022

Copy link
Copy Markdown
Contributor Author

/publish connector=connectors/source-mysql run-tests=false

🕑 Publishing the following connectors:
connectors/source-mysql
https://github.com/airbytehq/airbyte/actions/runs/3236373815


Connector Did it publish? Were definitions generated?
connectors/source-mysql

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@akashkulk

Akash Kulkarni (akashkulk) commented Oct 12, 2022

Copy link
Copy Markdown
Contributor Author

/publish connector=connectors/source-mysql-strict-encrypt run-tests=false

🕑 Publishing the following connectors:
connectors/source-mysql-strict-encrypt
https://github.com/airbytehq/airbyte/actions/runs/3236375145


Connector Did it publish? Were definitions generated?
connectors/source-mysql-strict-encrypt

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@akashkulk Akash Kulkarni (akashkulk) changed the title 🎉 Expose serverTimezone debezium option via MySQL Source spec for CDC 🎉 MySQL Source : Expose serverTimezone debezium option via MySQL Source spec for CDC Oct 12, 2022
@github-actions

Copy link
Copy Markdown
Contributor

NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to run corresponding integration tests:

  • source-mysql-strict-encrypt
  • source-mysql

@akashkulk
Akash Kulkarni (akashkulk) temporarily deployed to more-secrets October 12, 2022 16:51 Inactive
@github-actions

Copy link
Copy Markdown
Contributor

NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to run corresponding integration tests:

  • source-mysql-strict-encrypt
  • source-mysql

@akashkulk
Akash Kulkarni (akashkulk) merged commit e1701c9 into master Oct 12, 2022
@akashkulk
Akash Kulkarni (akashkulk) deleted the akash/cdc-tz branch October 12, 2022 16:53
@akashkulk
Akash Kulkarni (akashkulk) temporarily deployed to more-secrets October 12, 2022 16:53 Inactive
Johan Hammarstedt (jhammarstedt) pushed a commit to jhammarstedt/airbyte that referenced this pull request Oct 31, 2022
…ce spec for CDC (airbytehq#17815)

* Expose serverTimezone debezium option via MySQL Source spec for CDC

* Fix tests

* Bump dockerfile version

* Update Dockerfile

* Update mysql.md

* Update mysql.md

* Documentation

* Code Review Comments

* Addressing comments

* Addressing comments

* Addressing comments

* auto-bump connector version [ci skip]

Co-authored-by: Octavia Squidington III <octavia-squidington-iii@users.noreply.github.com>
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.

Expose serverTimezone debezium option via MySQL Source spec for CDC

5 participants