Re: Simplify some logical replication worker type checking - Mailing list pgsql-hackers

From Peter Smith
Subject Re: Simplify some logical replication worker type checking
Date
Msg-id CAHut+Ps_b=523-o-om21xmgYu5OrcNwg7QywA16ps+oRaopivA@mail.gmail.com
Whole thread Raw
In response to RE: Simplify some logical replication worker type checking  ("Zhijie Hou (Fujitsu)" <houzj.fnst@fujitsu.com>)
List pgsql-hackers
On Tue, Aug 1, 2023 at 12:59 PM Zhijie Hou (Fujitsu)
<houzj.fnst@fujitsu.com> wrote:
>
>
> About 2,3,4, it seems you should use "if (am_leader_apply_worker())" instead of
> "if (!am_leader_apply_worker())" because only leader apply worker should invoke
> this function.
>

Hi Hou-san,

Thanks for your review!

Oops. Of course, you are right. My cut/paste typo was mostly confined
to the post, but there was one instance also in the patch.

Fixed in v2.

------
Kind Regards,
Peter Smith.
Fujitsu Australia

Attachment

pgsql-hackers by date:

Previous
From: Jeff Davis
Date:
Subject: Re: Faster "SET search_path"
Next
From: Masahiro Ikeda
Date:
Subject: Fix pg_stat_reset_single_table_counters function