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][flink] introduce RecordAttributesProcessor for write operators #3791

Merged

Conversation

yunfengzhou-hub
Copy link
Contributor

Purpose

This PR introduces RecordAttributesProcessor for write operators, providing the ability to handle record attributes during constructing sink operators.

Tests

WriteOperatorTest.testProcessRecordAttributes is introduced to verify the correctness of this PR.

API and Format

This change does not affect API or storage format.

Documentation

This change does not introduce changes that require a document.

@yunfengzhou-hub yunfengzhou-hub force-pushed the record-attributes-processor branch 2 times, most recently from ec69835 to 060b335 Compare July 22, 2024 07:36
@yunfengzhou-hub
Copy link
Contributor Author

Hi @JingsongLi could you please take a look at this PR?

@yunfengzhou-hub yunfengzhou-hub force-pushed the record-attributes-processor branch 6 times, most recently from ed499cf to e17bb0b Compare July 29, 2024 14:07
@yunfengzhou-hub yunfengzhou-hub force-pushed the record-attributes-processor branch from e17bb0b to ed499cf Compare July 31, 2024 05:58
@yunfengzhou-hub yunfengzhou-hub force-pushed the record-attributes-processor branch 2 times, most recently from 759435a to 6a744a8 Compare August 12, 2024 05:31
@yunfengzhou-hub yunfengzhou-hub force-pushed the record-attributes-processor branch 2 times, most recently from fc03249 to 93b6816 Compare September 22, 2024 07:46
@yunfengzhou-hub yunfengzhou-hub force-pushed the record-attributes-processor branch from c00f164 to fd345bf Compare September 23, 2024 04:02
Copy link
Contributor

@JingsongLi JingsongLi left a comment

Choose a reason for hiding this comment

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

+1

@JingsongLi JingsongLi merged commit 236e38a into apache:master Oct 9, 2024
10 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.

2 participants