Re: Patch to add regression tests for SCHEMA

From: Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>
To: Robins Tharakan <tharakan(at)gmail(dot)com>
Cc: "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Patch to add regression tests for SCHEMA
Date: 2013-06-26 02:47:40
Message-ID: alpine.DEB.2.02.1306260439490.27845@localhost6.localdomain6
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


> Please find attached an updated patch with the said changes. I'll try to
> update the other patches (if they pertain to this feedback) and update
> on their respective threads (as well as on Commitfest).

Ok, this new version addresses my questions.

The patch works for me (nothing to compile, the added tests pass).

I recommend its inclusion as it tests corner cases especially about
permissions and error conditions, some of which may have security
implications if they were to fail some day. So this is a good thing.

The above remark applies to all systematic but not redundant regression
tests submitted. If the overall test was to be deemed too long and slow
for developers, I would suggest to have a two-tier system with basic and
fast tests for devs and longer tests for the build farm.

--
Fabien.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2013-06-26 02:55:25 Re: [RFC] Minmax indexes
Previous Message Mark Wong 2013-06-26 02:19:58 Re: [Review] Add SPI_gettypmod() to return a field's typemod from a TupleDesc