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

Replication origin support #204

Closed
wants to merge 2 commits into from
Closed

Conversation

pyrliu
Copy link
Contributor

@pyrliu pyrliu commented Dec 7, 2023

purpose of this PR is to add support for --replication-origin=foo that defines the replication origin setting to target database. This allows other sync/replication mechanisms to understand when data comes from pgsync and allows DBAs to configure setups for example to prevent triggering a cascade effect when pgsync is used with databases that have replication setup.

@pyrliu pyrliu changed the title Replication origin support Draft: Replication origin support Dec 7, 2023
@pyrliu pyrliu marked this pull request as draft December 7, 2023 06:38
@pyrliu pyrliu changed the title Draft: Replication origin support Replication origin support Dec 7, 2023
@pyrliu
Copy link
Contributor Author

pyrliu commented Dec 8, 2023

@ankane At this stage I would ask for your opinion for this PR and little help.

We use pgsync extensively and have recognized that we have need for this kind of feature. Is this something that you would accept as part of the featureset of your project?

If no, that is completely understandable and we will continue with our internal fork.

If yes, then I would gladly receive guidance that is this implementation okay for you and I humbly ask for little help on how this should made to work with Ruby2.7 + Postgres 9.6 😅

@ankane
Copy link
Owner

ankane commented Dec 9, 2023

Hi @pyrliu, thanks for the PR. Since this doesn't seem very common, I think it's better for a fork.

@ankane ankane closed this Dec 9, 2023
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