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

Feature/publish old image #25

Open
wants to merge 35 commits into
base: master
Choose a base branch
from

Conversation

samcorzineatfetch
Copy link

Ticket 🎟


What ⁉️

Adding support to publish the old image in dynamo streams that have both new and old images. This enables more complex flows for Dynamo CDC.

New field has been added to the payload named "old_document" which will be null if that field isn't there. This should be a backwards compatible change since no other fields have been modified.

Why ⁉️

This enables easy diffing of fields that have been modified.

How ⁉️

Added the field, tested changes manually on a dev cluster to verify behavior

How to review and test 📱

tbd


Checklist ✅

  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing tests are passing locally

breecoffey-fetch and others added 30 commits May 12, 2022 16:02
updated test-cases and jar filename as per build
Remove table hardcode - updated testcases and gradle build jar filename
updating format to JSON before pushing records to kafka and updated test cases
…JSON

updating format to JSON before pushing records to kafka
…JSON

Converting dyanmo documents to json before pushing to kafka
…JSON

updated to send unmarshalled record with schema to kafka topic
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