8000 Update pgcopydb sentinel's replay_lsn asynchronously. by dimitri · Pull Request #267 · dimitri/pgcopydb · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Update pgcopydb sentinel's replay_lsn asynchronously. #267 8000

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

Merged
merged 2 commits into from
Apr 28, 2023

Conversation

dimitri
Copy link
Owner
@dimitri dimitri commented Apr 27, 2023

When in follow replay mode, the pgcopydb sentinel is updated quite frequently so that we can see progress being made and be reactive about when to implement the migration cutover.

Now, we might have to consider some latency with the source database system where the replay_lsn is published, so rate-limiting those sentinel updates is important. Also, rate limiting alone might not be enough to prevent slowing down the replay process, so switch to using libpq async API.

See #254

dimitri added 2 commits April 27, 2023 17:16
When in follow replay mode, the pgcopydb sentinel is updated quite
frequently so that we can see progress being made and be reactive about when
to implement the migration cutover.

Now, we might have to consider some latency with the source database system
where the replay_lsn is published, so rate-limiting those sentinel updates
is important. Also, rate limiting alone might not be enough to prevent
slowing down the replay process, so switch to using libpq async API.
@dimitri dimitri force-pushed the fix/update-sentinel-asynchronously branch from cf4baf1 to e6a2266 Compare April 27, 2023 15:16
@dimitri dimitri merged commit 18ff38a into main Apr 28, 2023
@dimitri dimitri deleted the fix/update-sentinel-asynchronously branch April 28, 2023 08:56
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.

1 participant
0