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

[cdc] Added pulsar database cdc synchronization action #2345

Merged
merged 13 commits into from
Nov 30, 2023

Conversation

zhuangchong
Copy link
Contributor

Purpose

Added pulsar database cdc synchronization action and database synchronization base class .

Tests

API and Format

Documentation

@zhuangchong zhuangchong force-pushed the cdc-add-pulsar-database branch from 1c20f7c to a4a29f1 Compare November 19, 2023 03:44
@zhuangchong zhuangchong force-pushed the cdc-add-pulsar-database branch from d28e510 to b52a2d3 Compare November 20, 2023 05:29
@zhuangchong zhuangchong force-pushed the cdc-add-pulsar-database branch from 751b76f to b259cbb Compare November 21, 2023 01:25
@zhuangchong zhuangchong changed the title [cdc] Added pulsar database cdc synchronization action and database synchronization base class [cdc] Added pulsar database cdc synchronization action Nov 21, 2023
Copy link
Contributor

@yuzelin yuzelin left a comment

Choose a reason for hiding this comment

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

Thanks for contributing. Left some comments.

@zhuangchong
Copy link
Contributor Author

Thanks for contributing. Left some comments.

Thanks @yuzelin. done.

@zhuangchong zhuangchong requested a review from yuzelin November 30, 2023 04:46
Copy link
Contributor

@yuzelin yuzelin left a comment

Choose a reason for hiding this comment

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

+1

@yuzelin yuzelin merged commit f977afa into apache:master Nov 30, 2023
10 checks passed
@zhuangchong zhuangchong deleted the cdc-add-pulsar-database branch November 30, 2023 06:24
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