On Sat, Mar 28, 2020 at 7:08 PM Julien Rouhaud <rjuju...@gmail.com> wrote: > > On Sat, Mar 28, 2020 at 04:14:04PM +0530, Amit Kapila wrote: > > > > 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? > > I didn't tested those, and I'm not even sure how to properly and reliably test > that. Do you have any advice on how to achieve that? > > However the patch is mimicking the buffer instrumentation that already exists, > and the approach also looks correct to me. Do you have a reason to believe > that the approach that works for buffer usage wouldn't work for WAL records? > (I > of course agree that this should be tested anyway) >
The buffer usage infrastructure is for read-only queries (for ex. for stats like blks_hit, blks_read). As far as I can think, there is no easy way to test the WAL usage via that API. It might or might not be required in the future depending on whether we decide to use the same infrastructure for parallel writes. I think for now we should remove that part of changes and rather think how to get that for parallel operations that can write WAL. For ex. we might need to do something similar to what this patch has done in begin_parallel_vacuum and end_parallel_vacuum. Would you like to attempt that? -- With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com