Skip to content

Feature change tracking#1539

Open
Rayan (hichambouzkraoui) wants to merge 6 commits intoconfluentinc:masterfrom
hichambouzkraoui:feature-change_tracking
Open

Feature change tracking#1539
Rayan (hichambouzkraoui) wants to merge 6 commits intoconfluentinc:masterfrom
hichambouzkraoui:feature-change_tracking

Conversation

@hichambouzkraoui
Copy link
Copy Markdown

@hichambouzkraoui Rayan (hichambouzkraoui) commented Sep 15, 2025

Problem

I was trying to sync and old SQL Server version which only supports Change Tracking but not CDC to PostgreSQL
I couldn't find any Kafka source connector which support this feature
I found an old issue open for the same problem
refer to issue 80:(#80)

Solution

Extend kafka-connect-jdbc connect by adding new mode=change_tracking

Does this solution apply anywhere else?
  • [x ] yes
  • no
If yes, where?

issue 80:(#80)

Test Strategy

Done manual Testing by Setting up docker-compose with 3 services sqlserver, kafka,connect

Testing done:
  • [ x] Unit tests
  • [ x] Integration tests
  • System tests
  • [x ] Manual tests

Release Plan

@confluent-cla-assistant
Copy link
Copy Markdown

confluent-cla-assistant bot commented Sep 15, 2025

🎉 All Contributor License Agreements have been signed. Ready to merge.
✅ hichambouzkraoui
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.

@hichambouzkraoui Rayan (hichambouzkraoui) marked this pull request as ready for review September 15, 2025 15:24
@hichambouzkraoui Rayan (hichambouzkraoui) requested a review from a team as a code owner September 15, 2025 15:24
@heinebold
Copy link
Copy Markdown

I just stumbled over this because we need it too. Rayan (@hichambouzkraoui), do you have plans to continue with this?

@hichambouzkraoui Rayan (hichambouzkraoui) marked this pull request as draft January 21, 2026 05:47
@hichambouzkraoui
Copy link
Copy Markdown
Author

I just stumbled over this because we need it too. Rayan (@hichambouzkraoui), do you have plans to continue with this?

Matthias Brückner (@heinebold) I remember after signing the CLA the check was still pending, I'll retest the PR locally tomorrow and let you know

@hichambouzkraoui Rayan (hichambouzkraoui) marked this pull request as ready for review January 22, 2026 00:57
@hichambouzkraoui
Copy link
Copy Markdown
Author

Matthias Brückner (@heinebold)
I retested locally the PR and it's working fine

@heinebold
Copy link
Copy Markdown

Cool, thanks! Let's see if we get this upstream :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants