Skip to content

Conversation

RyanSkraba
Copy link
Contributor

Adds a new transformation that appends the key or selected key fields into the existing value.

This is similar to the existing standard Kafka transform ValueToKey, which replaces the existing key with fields from the value).

@RyanSkraba RyanSkraba requested a review from a team as a code owner April 25, 2025 07:39
@RyanSkraba RyanSkraba requested a review from Claudenw April 25, 2025 07:40
@RyanSkraba RyanSkraba force-pushed the rskraba/KCON-136-keytovalue branch from 22b4077 to d081c4b Compare April 25, 2025 17:01
Copy link
Contributor

@Claudenw Claudenw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good start.

@RyanSkraba RyanSkraba requested a review from Claudenw May 2, 2025 15:14
Copy link
Contributor

@Claudenw Claudenw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you want to rename the file I'll wait for merge otherwise let me know and I will push the button.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be KeyToValueConnectorTest rather than TestKeyToValueConnector?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ugh, this isn't a test :/

It needs a better name, or even optimised with a better strategy for all integration tests... Do you have any suggestions for the name for now? MockKeyToValueSource isn't quite right either :/

@Claudenw Claudenw merged commit 8986ea7 into master May 8, 2025
4 checks passed
@Claudenw Claudenw deleted the rskraba/KCON-136-keytovalue branch May 8, 2025 07:51
@RyanSkraba RyanSkraba mentioned this pull request May 9, 2025
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