Re: [PATCH] Add a test for pg_get_functiondef()

Поиск
Список
Период
Сортировка
От Abhijit Menon-Sen
Тема Re: [PATCH] Add a test for pg_get_functiondef()
Дата
Msg-id 20090421104604.GA7000@toroid.org
обсуждение исходный текст
Ответ на Re: [PATCH] Add a test for pg_get_functiondef()  (Tom Lane <tgl@sss.pgh.pa.us>)
Список 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 по дате отправления:

Предыдущее
От: vacuum@quantentunnel.de
Дата:
Сообщение: 8.4 semi-join slows down query performance (EXISTS)
Следующее
От: Simon Riggs
Дата:
Сообщение: Re: New trigger option of pg_standby