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

Update common version to fix record reporting #722

Merged
merged 1 commit into from Feb 22, 2024

Conversation

naveenmall11
Copy link
Member

Problem

https://confluent.slack.com/archives/C06HJ2VFQ5T
While projecting we were changing instance of record from InternalSinkRecord to SinkRecord and while reporting we were using modified records which was causing the original schema of messages to be changed while serialising again.

Solution

Update version of common to 11.2.5.
confluentinc/kafka-connect-storage-common#347

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

Test Strategy

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

Release Plan

@naveenmall11 naveenmall11 requested a review from a team as a code owner February 21, 2024 11:46
Copy link

cla-assistant bot commented Feb 21, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

1 similar comment
Copy link

cla-assistant bot commented Feb 21, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@naveenmall11 naveenmall11 changed the base branch from master to 10.0.x February 21, 2024 11:47
@naveenmall11 naveenmall11 merged commit 15ce49d into 10.0.x Feb 22, 2024
2 of 3 checks passed
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.

None yet

2 participants