On Thu, Mar 4, 2021 at 5:24 PM Greg Nancarrow <gregn4...@gmail.com> wrote: > > On Thu, Mar 4, 2021 at 10:07 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > > > > >Also, in > > standard_planner, we should add these partitionOids only for > > parallel-mode. > > > > It is doing that in v20 patch (what makes you think it isn't?). >
The below code snippet: + /* For AcquireExecutorLocks(). */ + if (IsModifySupportedInParallelMode(parse->commandType)) + result->partitionOids = glob->partitionOids; I understand that you have a check for the parallel mode in AcquireExecutorLocks but why can't we have it before adding that to planned statement -- With Regards, Amit Kapila.
