Re: Passing CopyMultiInsertInfo structure to CopyMultiInsertInfoNextFreeSlot()

From: David Rowley <david(dot)rowley(at)2ndquadrant(dot)com>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: Ashutosh Sharma <ashu(dot)coek88(at)gmail(dot)com>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com>
Subject: Re: Passing CopyMultiInsertInfo structure to CopyMultiInsertInfoNextFreeSlot()
Date: 2019-05-18 01:14:07
Message-ID: CAKJS1f96xwiU6vtePaxwq55u7AArSQDNew2W6peYH88EmSnFTA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sat, 18 May 2019 at 12:49, Andres Freund <andres(at)anarazel(dot)de> wrote:
>
> On 2019-05-18 06:14:15 +0530, Ashutosh Sharma wrote:
> > I actually feel that the function name itself is not correct here, it
> > appears to be confusing and inconsistent considering the kind of work that
> > it is doing. I think, the function name should have been CopyMultiInsert
> > *Buffer*NextFreeSlot() instead of CopyMultiInsert*Info*NextFreeSlot(). What
> > do you think, Andres, David, Alvaro ?
>
> Unless somebody else presses back hard against doing so *soon*, I'm
> going to close this open issue. I don't think it's worth spending
> further time arguing about a few characters.

I'd say if we're not going to bother removing the unused param that
there's not much point in renaming the function. The proposed name
might make sense if the function was:

static inline TupleTableSlot *
CopyMultiInsertBufferNextFreeSlot(CopyMultiInsertBuffer *buffer, Relation rel)

then that might be worth a commit, but giving it that name without
changing the signature to that does not seem like an improvement to
me.

I'm personally about +0.1 for making the above change, which is well
below my threshold for shouting and screaming.

--
David Rowley http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2019-05-18 01:18:16 Re: New vacuum option to do only freezing
Previous Message Tom Lane 2019-05-18 01:12:15 Re: Is it safe to ignore the return value of SPI_finish and SPI_execute?