This patch adds a regression test for the bug with domains and ALTER
TABLE that Tom fixed a few hours ago.

Barring any objections I'll apply this to HEAD by the end of the day.

-Neil



---------------------------(end of broadcast)---------------------------
TIP 8: explain analyze is your friend

Neil Conway <neilc@samurai.com> writes:
> This patch adds a regression test for the bug with domains and ALTER
> TABLE that Tom fixed a few hours ago.

It might be better to make the tables TEMP tables --- there are some
regression tests that depend on the set of existing tables, IIRC.
Otherwise ok.

regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 3: if posting/reading through Usenet, please send an appropriate
subscribe-nomail command to majordomo@postgresql.org so that your
message can get through to the mailing list cleanly

On Tue, 2005-01-25 at 11:56 +1100, Neil Conway wrote:
> This patch adds a regression test for the bug with domains and ALTER
> TABLE that Tom fixed a few hours ago.

Applied.

-Neil



---------------------------(end of broadcast)---------------------------
TIP 5: Have you checked our extensive FAQ?

http://www.postgresql.org/docs/faq

On Mon, 2005-01-24 at 22:22 -0500, Tom Lane wrote:
> It might be better to make the tables TEMP tables --- there are some
> regression tests that depend on the set of existing tables, IIRC.
> Otherwise ok.

True, although those kind of dependencies usually cause an obvious test
failure (which this patch doesn't cause). I've been thinking that
there's some value in keeping database objects creating by regression
tests around, so that we get better pg_dump coverage (a quick and dirty
pg_dump test is to dump the database created by "make installcheck" and
restore it). Admittedly this is pretty primitive, but it's better than
nothing.

[ Sorry for applying the patch already, I hadn't received your mail ]

-Neil



---------------------------(end of broadcast)---------------------------
TIP 7: don't forget to increase your free space map settings