On Thursday, November 18, 2021 2:08 PM Greg Nancarrow <gregn4...@gmail.com> wrote: > A minor comment: Thanks for your comments ! > doc/src/sgml/ref/alter_subscription.sgml > (1) disable_on_err? > > + <literal>disable_on_err</literal>. > > This doc update names the new parameter as "disable_on_err" instead of > "disable_on_error". > Also "disable_on_err" appears in a couple of the test case comments. Fixed all 3 places.
At the same time, I changed one function name from IsSubscriptionDisablingError() to IsTransientError() so that it can express what it checks correctly. Of course, the return value of true or false becomes reverse by this name change, but This would make the function more general. Also, its comments were fixed. This version also depends on the v23 of skip xid [1] [1] - https://www.postgresql.org/message-id/CAD21AoA5jupM6O%3DpYsyfaxQ1aMX-en8%3DQNgpW6KfXsg7_CS0CQ%40mail.gmail.com Best Regards, Takamichi Osumi
v6-0001-Optionally-disable-subscriptions-on-error.patch
Description: v6-0001-Optionally-disable-subscriptions-on-error.patch
