Re: [PATCH] Add a test for pg_get_functiondef() - Mailing list pgsql-hackers

From Abhijit Menon-Sen
Subject Re: [PATCH] Add a test for pg_get_functiondef()
Date
Msg-id 20090421104604.GA7000@toroid.org
Whole thread Raw
In response to Re: [PATCH] Add a test for pg_get_functiondef()  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
At 2009-04-13 10:40:41 -0400, tgl@sss.pgh.pa.us wrote:
>
> Abhijit Menon-Sen <ams@oryx.com> writes:
> > [ a test whose purpose he didn't bother to describe ]

I'm sorry about that.

> What is the value of this? It seems far more likely to cause
> maintenance pain than to catch anything interesting.

While I was writing pg_get_functiondef(), I found myself wishing for
just such a function (one with all the toppings, as it were) to make
sure I wasn't missing something. I thought it would be useful to keep
the function around to prevent inadvertent breakage, that's all. The
fact that the code was duplicated in pg_dump also worried me, though
this test case doesn't help with that.

What maintenance pain would it cause? It seems to me that if anything
causes this test to fail in future, it deserves a second look.

But it's a trivial patch, hardly worth spending much time discussing.
Perhaps its being in the mailing list archives is good enough.

-- ams


pgsql-hackers by date:

Previous
From: vacuum@quantentunnel.de
Date:
Subject: 8.4 semi-join slows down query performance (EXISTS)
Next
From: Simon Riggs
Date:
Subject: Re: New trigger option of pg_standby