On Tue, 14 May 2019 at 13:00, Michael Paquier <mich...@paquier.xyz> wrote: > > On Mon, May 13, 2019 at 08:17:49PM +0530, Ashutosh Sharma wrote: > > Thanks for the confirmation David. The patch looks good to me. > > It looks to me that it can be a matter a consistency with the other > APIs dealing with multi-inserts in COPY. For now I have added an open > item on that.
When I wrote the code I admit that I was probably wearing my object-orientated programming hat. I had in mind that the whole function series would have made a good class. Passing the CopyMultiInsertInfo was sort of the non-OOP equivalent to having this/Me/self available, as it would be for any instance method of a class. Back to reality, this isn't OOP, so I was wearing the wrong hat. I think the unused parameter should likely be removed. It's probably not doing a great deal of harm since the function is static inline and the compiler should be producing any code for the unused param, but for the sake of preventing confusion, it should be removed. Ashutosh had to ask about it, so it wasn't immediately clear what the purpose of it was. Since there's none, be gone with it, I say. -- David Rowley http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services
