Re: Patch: clean up addRangeTableEntryForFunction - Mailing list pgsql-hackers

From Etsuro Fujita
Subject Re: Patch: clean up addRangeTableEntryForFunction
Date
Msg-id 005b01ce77c1$77215630$65640290$@lab.ntt.co.jp
Whole thread Raw
In response to Re: Patch: clean up addRangeTableEntryForFunction  (Robert Haas <robertmhaas@gmail.com>)
List pgsql-hackers
Robert Haas wrote:
> On Tue, Jan 22, 2013 at 11:45 AM, David Fetter <david@fetter.org> wrote:
> > I've been working with Andrew Gierth (well, mostly he's been doing the
> > work, as usual) to add WITH ORDINALITY as an option for set-returning
> > functions.  In the process, he found a minor opportunity to clean up
> > the interface for $SUBJECT, reducing the call to a Single Point of
> > Truth for lateral-ness, very likely improving the efficiency of calls
> > to that function.
> >
> > Please find attached the patch.
> 
> I think this patch is utterly pointless.  I recommend we reject it.
> If this were part of some larger refactoring that was going in some direction
> we could agree on, it might be worth it.  But as it is, I think it's just a
> shot in the dark whether this change will end up being better or worse, and
> my personal bet is that it won't make any difference whatsoever.

To be frank, I agree with Robert.

Sorry for the delay in my review.

Best regards,
Etsuro Fujita




pgsql-hackers by date:

Previous
From: Simon Riggs
Date:
Subject: Re: Move unused buffers to freelist
Next
From: Kyotaro HORIGUCHI
Date:
Subject: Re: Reduce maximum error in tuples estimation after vacuum.