Re: Add regression tests for SET xxx
От | Szymon Guz |
---|---|
Тема | Re: Add regression tests for SET xxx |
Дата | |
Msg-id | CAFjNrYtMg3cxn9Zx5rXfpRmMbVAuCkuy-1xU4WhSnEQBtDmsww@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: Add regression tests for SET xxx (Robins Tharakan <tharakan@gmail.com>) |
Ответы |
Re: Add regression tests for SET xxx
Re: Add regression tests for SET xxx Re: Add regression tests for SET xxx |
Список | pgsql-hackers |
On 18 June 2013 02:33, Robins Tharakan <tharakan@gmail.com> wrote:
Thanks !PFA the updated patch. Also remove a trailing whitespace at the end of SQL script.--Robins TharakanOn 17 June 2013 17:29, Szymon Guz <mabewlun@gmail.com> wrote:On 26 May 2013 19:56, Robins Tharakan <tharakan@gmail.com> wrote:--Hi,Please find attached a patch to take code-coverage of SET (SESSION / SEED / TRANSACTION / DATESTYLE / TIME ZONE) (src/backend/commands/variable.c) from 65% to 82%.Any and all feedback is welcome.--Robins Tharakan
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackersHi,the patch applies cleanly on code from trunk, however there are failing tests, diff attached.regardsSzymon
Hi,
I've checked the patch. Applies cleanly. Tests pass this time :)
Could you add me as a reviewer to commitfest website, set this patch a reviewed and add this email to the patch history?
I cannot login to the commitfest app, there is some bug with that.
thanks,
Szymon Guz
В списке pgsql-hackers по дате отправления: