Re: [COMMITTERS] pgsql: Add notion of a "transform function" that can simplify function

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.

In response to

Responses

Browse pgsql-committers by date

  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

Browse pgsql-hackers by date

  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