Re: Skipping logical replication transactions on subscriber side
Amit Kapila <amit.kapila16@gmail.com>
On Thu, May 27, 2021 at 12:01 PM Masahiko Sawada <sawada.mshk@gmail.com> wrote: > > On Wed, May 26, 2021 at 6:11 PM Amit Kapila <amit.kapila16@gmail.com> wrote: > > > > I agree with you that specifying XID could be easier and > > understandable for users. I was thinking and studying a bit about what > > other systems do in this regard. Why don't we try to provide conflict > > resolution methods for users? The idea could be that either the > > conflicts can be resolved automatically or manually. In the case of > > manual resolution, users can use the existing methods or the XID stuff > > you are proposing here and in case of automatic resolution, the > > in-built or corresponding user-defined functions will be invoked for > > conflict resolution. There are more details to figure out in the > > automatic resolution scheme but I see a lot of value in doing the > > same. > > Yeah, I also see a lot of value in automatic conflict resolution. But > maybe we can have both ways? For example, in case where the user wants > to resolve conflicts in different ways or a conflict that cannot be > resolved by automatic resolution (not sure there is in practice > though), the manual resolution would also have value. > Right, that is exactly what I was saying. So, even if both can be done as separate patches, we should try to design the manual resolution in a way that can be extended for an automatic resolution system. I think we can try to have some initial idea/design/POC for an automatic resolution as well to ensure that the manual resolution scheme can be further extended. -- With Regards, Amit Kapila.
Commits
-
Test ALIGNOF_DOUBLE==4 compatibility under ALIGNOF_DOUBLE==8.
- c1da0acbb06e 15.0 landed
-
Reorder subskiplsn in pg_subscription to avoid alignment issues.
- 79b716cfb7a1 15.0 landed
-
Add ALTER SUBSCRIPTION ... SKIP.
- 208c5d65bbd6 15.0 landed
-
Optionally disable subscriptions on error.
- 705e20f8550c 15.0 cited
-
Update docs of logical replication for commit 8d74fc96db.
- 85c61ba8920b 15.0 landed
-
Respect permissions within logical replication.
- a2ab9c06ea15 15.0 cited
-
Fix regression test failure caused by commit 8d74fc96db.
- 41e66fee0516 15.0 landed
-
Add a view to show the stats of subscription workers.
- 8d74fc96db5f 15.0 landed
-
Add logical change details to logical replication worker errcontext.
- abc0910e2e0a 15.0 landed
-
Rename LOGICAL_REP_MSG_STREAM_END to LOGICAL_REP_MSG_STREAM_STOP.
- 4cd7a1896871 15.0 landed
-
Fix typo in protocol.sgml.
- e1915646658d 14.0 landed
- 0ac1aee0d7d8 15.0 landed
-
Remove unused argument in apply_handle_commit_internal().
- f4b939f1a372 14.0 landed
- 16bd4becee32 15.0 landed
-
Fix replication of in-progress transactions in tablesync worker.
- 0926e96c4934 14.0 cited
-
Reorder pg_sequence columns to avoid alignment issue
- f3b421da5f4a 10.0 cited