Re: Patch to add regression tests for SCHEMA

Поиск
Список
Период
Сортировка
От Robert Haas
Тема Re: Patch to add regression tests for SCHEMA
Дата
Msg-id CA+TgmoZBRkjdmEd_PQNL-19FiGoCn8OHzQVsuFXBKsvJSJmpNw@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Patch to add regression tests for SCHEMA  (Robins Tharakan <tharakan@gmail.com>)
Ответы Re: Patch to add regression tests for SCHEMA  (Robins Tharakan <tharakan@gmail.com>)
Список pgsql-hackers
On Wed, May 8, 2013 at 9:19 PM, Robins Tharakan <tharakan@gmail.com> wrote:
> 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).

This patch updates the parallel schedule but not the serial schedule.
Please try to remember to update both files when adding new test
files.  Also, please observe the admonitions in the parallel schedule,
at top of file:

# By convention, we put no more than twenty tests in any one parallel group;
# this limits the number of connections needed to run the tests.

In this particular case, I think that adding a new set of tests isn't
the right thing anyway.  Schemas are also known as namespaces, and the
existing "namespace" test is where related test cases live.  Add these
tests there instead.

Rename regression users to regress_rol_nsp1 etc. per convention
established in the CREATE OPERATOR patch.

Setting patch to "Waiting on Author".

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Bruce Momjian
Дата:
Сообщение: Re: possible/feasible to specify field and value in error msg?
Следующее
От: Andres Freund
Дата:
Сообщение: Re: refresh materialized view concurrently