From: | Abhijit Menon-Sen <ams(at)oryx(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: [PATCH] Add a test for pg_get_functiondef() |
Date: | 2009-04-21 10:46:04 |
Message-ID: | 20090421104604.GA7000@toroid.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
At 2009-04-13 10:40:41 -0400, tgl(at)sss(dot)pgh(dot)pa(dot)us wrote:
>
> Abhijit Menon-Sen <ams(at)oryx(dot)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
From | Date | Subject | |
---|---|---|---|
Next Message | Simon Riggs | 2009-04-21 11:03:56 | Re: New trigger option of pg_standby |
Previous Message | Albe Laurenz | 2009-04-21 10:36:44 | Re: trouble with to_char('L') |