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

DBZ-7363 Debezium stops producing events on schema changes in cassandra4 #118

Merged
merged 2 commits into from
Feb 21, 2024

Conversation

schampilomatis
Copy link
Contributor

Adjust cassandra4 schema listener to use old table metadata Table ID when replacing the metadata in Cassandra 4 Schema Provider. Replace current logic of calculating the uuid from keyspace name and table name which does not match.

Copy link

Hi @schampilomatis. Thank you for your valuable contribution.
Please author your commit(s) using an email linked to your GitHub account.

Copy link

Welcome as a new contributor to Debezium, @schampilomatis. Reviewers, please add missing author name(s) and alias name(s) to the COPYRIGHT.txt and Aliases.txt respectively.

@jpechane
Copy link
Contributor

@schampilomatis Thanks for the PR. Could you please also add a test that will verify the new behaviour if possible? Thanks

Copy link

Hi @schampilomatis, thanks for your contribution. Please prefix the commit message(s) with the DBZ-xxx JIRA issue key.

@schampilomatis
Copy link
Contributor Author

@jpechane Sorry for the late response, I was away for a few weeks. I added the test to verify the behaviour.

@schampilomatis schampilomatis force-pushed the DBZ-7363 branch 2 times, most recently from d8bcfc7 to be9773d Compare February 15, 2024 12:15
@schampilomatis
Copy link
Contributor Author

@jpechane I updated the branch with formatting, can you retrigger the workflow?

Adjust cassandra4 schema listener to use old table metadata Table ID
when replacing the metadata in Cassandra 4 Schema Provider. Replace
current logic of calculating the uuid from keyspace name and table name
which does not match. Add SchemaChangeListenerTest to verify the
behavior.
@jpechane
Copy link
Contributor

I pushed a commit with copyright header

@jpechane jpechane merged commit 649b861 into debezium:main Feb 21, 2024
3 checks passed
@jpechane
Copy link
Contributor

@schampilomatis Applied, thanks a lot!

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