On Fri, May 12, 2023 at 9:10 AM Masahiko Sawada <sawada.m...@gmail.com> wrote: > > On Fri, May 12, 2023 at 1:12 AM Robert Haas <robertmh...@gmail.com> wrote: > > > > On Thu, May 11, 2023 at 7:38 AM Amit Kapila <amit.kapil...@gmail.com> wrote: > > > Do we want the initial sync to also respect 'run_as_owner' option? I > > > might be missing something but I don't see anything in the docs about > > > initial sync interaction with this option. In the commit a2ab9c06ea, > > > we did the permission checking during the initial sync so I thought we > > > should do it here as well. > > > > It definitely should work that way. lf it doesn't, that's a bug. > > After some tests, it seems that the initial sync worker respects > 'run_as_owner' during catching up but not during COPYing. >
Yeah, I was worried during copy phase only. During catchup, the code is common with apply worker code, so it will work. -- With Regards, Amit Kapila.