On Mon, Jul 1, 2013 at 5:59 PM, Robins Tharakan <thara...@gmail.com> wrote:
> > Thanks Marko for pointing out about guc.sql. > > Please find attached a patch to move DISCARD related tests from guc.sql to > discard.sql. It adds an extra test for a DISCARD PLANS line, although I > amn't sure on how to validate that its working. > > Personally, I wouldn't call this a great patch, since most of the tests > were already running, although in a generic script. The separation of > DISCARD related tests to another file is arguably good for the long-term > though. > Robins, You must add this new test case called "discard" to src/test/regress/parallel_schedule and src/test/regress/serial_schedule, because if we do "make check" the new "discard" test case is not executed. Regards, -- Fabrízio de Royes Mello Consultoria/Coaching PostgreSQL >> Blog sobre TI: http://fabriziomello.blogspot.com >> Perfil Linkedin: http://br.linkedin.com/in/fabriziomello >> Twitter: http://twitter.com/fabriziomello