From: | Noah Misch <noah(at)leadboat(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: [COMMITTERS] pgsql: Add notion of a "transform function" that can simplify function |
Date: | 2012-03-23 15:31:47 |
Message-ID: | 20120323153147.GA1803@tornado.leadboat.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On Fri, Mar 23, 2012 at 10:55:52AM -0400, Tom Lane wrote:
> Robert Haas <rhaas(at)postgresql(dot)org> writes:
> > Add notion of a "transform function" that can simplify function calls.
>
> Why exactly was this thought to be a good idea:
>
> > * A NULL original expression disables use of transform functions while
> > * retaining all other behaviors.
We last spoke of that idea here, albeit in minimal detail:
http://archives.postgresql.org/pgsql-hackers/2011-06/msg00918.php
> AFAICT that buys nothing except to greatly complicate the API
> specification for simplify_function, something that is now proving
> problematic for Marti's requested refactoring [1]. If it's
> inappropriate for a transform function to modify a CoerceViaIO call,
> surely the transform function can be expected to know that.
I did it that way because it looked wrong to pass the same CoerceViaIO node to
transforms of both the input and output functions. Thinking about it again
now, doing so imposes no fundamental problems. Feel welcome to change it.
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2012-03-23 15:31:54 | Re: [COMMITTERS] pgsql: Add notion of a "transform function" that can simplify function |
Previous Message | Robert Haas | 2012-03-23 15:25:58 | Re: [COMMITTERS] pgsql: Add notion of a "transform function" that can simplify function |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2012-03-23 15:31:54 | Re: [COMMITTERS] pgsql: Add notion of a "transform function" that can simplify function |
Previous Message | Billy Earney | 2012-03-23 15:28:48 | query cache |