On Sat, Mar 28, 2020 at 12:54 AM Julien Rouhaud <rjuju...@gmail.com> wrote: > > On Fri, Mar 27, 2020 at 8:21 PM Kirill Bychik <kirill.byc...@gmail.com> wrote: > > > > > > All these are really valuable objections. Unfortunately, I won't be > > able to get all sorted out soon, due to total lack of time. I would be > > very glad if somebody could step in for this patch. > > I'll try to do that tomorrow! >
I see some basic problems with the patch. The way it tries to compute WAL usage for parallel stuff doesn't seem right to me. Can you share or point me to any test done where we have computed WAL for parallel operations like Parallel Vacuum or Parallel Create Index? Basically, I don't know changes done in ExecInitParallelPlan and friends allow us to compute WAL for parallel operations. Those will primarily cover parallel queries that won't write WAL. How you have tested those changes? -- With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com
