Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Topic, message and key from columns #1231

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

walec51
Copy link

@walec51 walec51 commented Aug 8, 2022

Problem

We wanted to use this JDBC Connector not to produce messages that expose our database schema but clean JSON messages that have been prepared in a database table after a transaction was commited.

We also wanted those JSON messages also indicate a destination topic and key on a message to message basis.

Thanks to this we can implement this pattern:

https://vimeo.com/111998645?embedded=true&source=video_title&owner=19072516

using this connector

Solution

Adding some configuration options with which we can indicate which table columns hold:

  • the destination topic
  • the messages value
  • the messages key
Does this solution apply anywhere else?
  • yes
  • no
If yes, where?

Every system this whats to implement this pattern using this connector:

https://vimeo.com/111998645?embedded=true&source=video_title&owner=19072516

Test Strategy

Testing done:
  • Unit tests
  • Integration tests
  • System tests
  • Manual tests

Release Plan

We would like to merge this to master.

We are aware that this PR still needs some work and would be grateful for some early feedback and commends.

We develop this code with @Mistral and @Lukier1 and we are already using it on our test environments at ewejsciowki.pl

@CLAassistant
Copy link

CLAassistant commented Aug 8, 2022

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 3 committers have signed the CLA.

❌ Lukier1
❌ Mistral
❌ Adam Walczak


Adam Walczak seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@OneCricketeer
Copy link

Ideally, you'd use a series of custom transformers to achieve this same result. Such as ValueToKey, and one that overrides the ConnectRecord partition, etc.

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.

5 participants