-
Notifications
You must be signed in to change notification settings - Fork 21
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
Logical Replication #899
base: main
Are you sure you want to change the base?
Logical Replication #899
Conversation
Codecov ReportAttention: Patch coverage is
❌ Your patch check has failed because the patch coverage (17.94%) is below the target coverage (33.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #899 +/- ##
==========================================
- Coverage 75.05% 70.10% -4.96%
==========================================
Files 13 14 +1
Lines 3704 4054 +350
Branches 548 625 +77
==========================================
+ Hits 2780 2842 +62
- Misses 718 999 +281
- Partials 206 213 +7 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code looks great! I'll only do some more testing tomorrow.
Co-authored-by: Marcelo Henrique Neppel <[email protected]>
Co-authored-by: Marcelo Henrique Neppel <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
This is a copy of VM PR.
This is an implementation of the Logical Replication specification (WIP): https://docs.google.com/document/d/1f84Z4cZwUD4q4bJsXTTC_PUaZr7pPB0Q6HnQ9cgCX_o/edit?usp=sharing.
Basic testing
postgresql
andpostgresql2
juju integrate postgresql:logical-replication-offer postgresql2:logical-replication
postgresql
:create table asd (message text); insert into asd values ('hello');
postgresql2
sidejuju run postgresql/leader add-publication name=asd database=postgres tables=public.asd
juju run postgresql2/leader subscribe name=asd
postgresql2
side:select * from asd;
postgresql2
subscriberKey notes
cluster_topology_change
observer as primary endpoint is resolved automatically and always points to the right unit.