Re: Regress tests to improve the function coverage of schemacmds and user and tablespace files

Поиск
Список
Период
Сортировка
От David Rowley
Тема Re: Regress tests to improve the function coverage of schemacmds and user and tablespace files
Дата
Msg-id CAApHDvqMHNU6yB4hpW4nfQ3nTUWuWTEQtm4cAqFPUjr1o9d_QQ@mail.gmail.com
обсуждение исходный текст
Ответ на Regress tests to improve the function coverage of schemacmds and user and tablespace files  (Haribabu kommi <haribabu.kommi@huawei.com>)
Ответы Re: Regress tests to improve the function coverage of schemacmds and user and tablespace files  (Kevin Grittner <kgrittn@ymail.com>)
Список pgsql-hackers
On Thu, Oct 24, 2013 at 9:59 PM, Haribabu kommi <haribabu.kommi@huawei.com> wrote:

 

Here I have added some more regression tests to improve the missing function coverage of

schemacmds.c, user.c and tablespace.c.

 

The added tests are mainly RENAME TO and OWNER TO support.


I've just had a look at both of these patches. All tests that have been added seem to cover new areas that are not previously tested, they also seem to cleanup properly after themselves, so I think these should be a worthwhile addition to the regression tests.

The only thing I can pickup on which is at fault is the the trailing white space 

src/test/regress/sql/privileges.sql:837: trailing whitespace.
+\c -

But I can't imagine it's worth submitting a new patch to fix it.

I've marked the patch as ready for commiter

Regards

David Rowley
 

 

patches are attached in the mail.

please check and provide your suggestions.

 

Regards,

Hari babu.



--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


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

Предыдущее
От: Vik Fearing
Дата:
Сообщение: Re: new unicode table border styles for psql
Следующее
От: Sawada Masahiko
Дата:
Сообщение: Re: segfault with contrib lo