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.

Sign up for GitHub

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

Jump to bottom

CDC support broadcast table #28999

Merged

sandynz merged 1 commit into apache:master from azexcy:broadcast-table

Nov 10, 2023

Contributor

azexcy commented Nov 10, 2023 •

edited

Loading

Related #22500.

Changes proposed in this pull request:

CDC support broadcast table

Before committing this PR, I'm sure that I have checked the following options:

My code follows the code of conduct of this project.
I have self-reviewed the commit code.
I have (or in comment I request) added corresponding labels for the pull request.
I have passed maven check locally : ./mvnw clean install -B -T1C -Dmaven.javadoc.skip -Dmaven.jacoco.skip -e.
I have made corresponding changes to the documentation.
I have added corresponding unit tests for my changes.


          CDC support broadcast table

azexcy added type: new feature feature: pipeline labels

azexcy added this to the 5.4.2 milestone

sandynz approved these changes

View reviewed changes

sandynz merged commit e66222a into apache:master

24 checks passed

Contributor

sandynz commented Nov 10, 2023

TODO: Add unit test for CDCDataNodeUtils, since t_address is changed to broadcast table from single table in E2E

azexcy deleted the broadcast-table branch

November 10, 2023 09:33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature: pipeline type: new feature