Le lun. 13 avr. 2020 à 13:47, Amit Kapila <amit.kapil...@gmail.com> a écrit :
> On Mon, Apr 13, 2020 at 1:10 PM Julien Rouhaud <rjuju...@gmail.com> wrote: > > > > On Mon, Apr 13, 2020 at 8:11 AM Amit Kapila <amit.kapil...@gmail.com> > wrote: > > > > > > On Sat, Apr 11, 2020 at 6:55 PM Julien Rouhaud <rjuju...@gmail.com> > wrote: > > > > > > > > On Fri, Apr 10, 2020 at 9:37 PM Julien Rouhaud <rjuju...@gmail.com> > wrote: > > > > > > > > > I tried to take into account all that have been discussed, but I have > > > > to admit that I'm absolutely not sure of what was actually decided > > > > here. I went with those changes: > > > > > > > > - rename wal_num_fpw to wal_fpw for consistency, both in pgss view > > > > fiel name but also everywhere in the code > > > > - change comments to consistently mention "full page writes > generated" > > > > - changed pgss and explain documentation to mention "full page images > > > > generated", from Justin's patch on another thread > > > > > > > > > > I think it is better to use "full page writes" to be consistent with > > > other places. > > > > > > > - kept "amount" of WAL bytes > > > > > > > > > > Okay, but I would like to make another change suggested by Justin > > > which is to replace "count" with "number" at a few places. > > > > Ah sorry I missed this one. +1 it also sounds better. > > > > > I have made the above two changes in the attached. Let me know what > > > you think about attached? > > > > It all looks good to me! > > > > Pushed. > Thanks a lot Amit! >
