Re: Add more regression tests for CREATE OPERATOR

Поиск
Список
Период
Сортировка
От Josh Berkus
Тема Re: Add more regression tests for CREATE OPERATOR
Дата
Msg-id 51CB3CAC.1050003@agliodbs.com
обсуждение исходный текст
Ответ на Add more regression tests for CREATE OPERATOR  (Robins Tharakan <tharakan@gmail.com>)
Ответы Re: Add more regression tests for CREATE OPERATOR  (Szymon Guz <mabewlun@gmail.com>)
Список pgsql-hackers
> Is it enough to provide the description in the commitfest app, or is that
> better to send an email and provide link in commitfest?

Better to do it here, on the list.

> This is a patch only with regression tests, is that enough to write
> something like: "This patch applies cleanly on trunk code. All tests pass,
> the test coverage increses as provided."? Or do you expect some more info?

Yes, mainly:

a) does it test what it purports to test?

b) do the tests pass on your machine?


-- 
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com



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

Предыдущее
От: Fabien COELHO
Дата:
Сообщение: Re: Add more regression tests for dbcommands
Следующее
От: Josh Berkus
Дата:
Сообщение: Re: Kudos for Reviewers -- straw poll