On Fri, Jul 3, 2020 at 5:18 PM Masahiko Sawada <masahiko.saw...@2ndquadrant.com> wrote: > > On Fri, 3 Jul 2020 at 17:07, vignesh C <vignes...@gmail.com> wrote: > > > > Hi, > > > > While checking through the code I found that some of the function > > parameters in reorderbuffer & vacuumlazy are not used. I felt this > > could be removed. I'm not sure if it is kept for future use or not. > > Attached patch contains the changes for the same. > > Thoughts? > > > > For the part of parallel vacuum change, it looks good to me. >
Unlike ReorderBuffer, this change looks fine to me as well. This is a quite recent (PG13) change and it would be good to remove it now. So, I will push this part of change unless I hear any objection in a day or so. -- With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com