Postgres Pro
Downloads
Home   >   mailing lists

Re: Add regression tests for SET xxx - Mailing list pgsql-hackers

From Robins Tharakan
Subject Re: Add regression tests for SET xxx
Date
Msg-id CAEP4nAx07zHF_3n0gPzYF4Yve3Tna3=oMZ=KnrEKCqMvTAWi0A@mail.gmail.com
Whole thread Raw
In response to Re: Add regression tests for SET xxx  (Szymon Guz <mabewlun@gmail.com>)
Responses Re: Add regression tests for SET xxx
List pgsql-hackers
Tree view
Thanks !

PFA the updated patch. Also remove a trailing whitespace at the end of SQL script.

--
Robins Tharakan


On 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-hackers


Hi,
the patch applies cleanly on code from trunk, however there are failing tests, diff attached.

regards
Szymon

Attachment

pgsql-hackers by date:

Previous
From: Josh Berkus
Date:
Subject: How do we track backpatches?
Next
From: Michael Paquier
Date:
Subject: Re: Support for REINDEX CONCURRENTLY