Re: Add more regression tests for dbcommands
От
Robins Tharakan
Тема
Re: Add more regression tests for dbcommands
Дата
Msg-id
CAEP4nAzKXnx0FLtWw32KX-nrhDf0SjyAu25JHev3s-vGPHJRtA@mail.gmail.com
Ответ на
Re: Add more regression tests for dbcommands (Andres Freund)
Список
Дерево обсуждения
Add more regression tests for dbcommands Robins Tharakan <tharakan@gmail.com>
Re: Add more regression tests for dbcommands Tom Lane <tgl@sss.pgh.pa.us>
Re: Add more regression tests for dbcommands Fabien COELHO <coelho@cri.ensmp.fr>
Re: Add more regression tests for dbcommands Tom Lane <tgl@sss.pgh.pa.us>
Re: Add more regression tests for dbcommands Fabien COELHO <coelho@cri.ensmp.fr>
Re: Add more regression tests for dbcommands Andres Freund <andres@2ndquadrant.com>
Re: Add more regression tests for dbcommands Fabien COELHO <fabien.coelho@mines-paristech.fr>
Re: Add more regression tests for dbcommands Robins Tharakan <tharakan@gmail.com>
Re: Add more regression tests for dbcommands Fabien COELHO <coelho@cri.ensmp.fr>
Re: Add more regression tests for dbcommands Robins Tharakan <tharakan@gmail.com>
Re: Add more regression tests for dbcommands Andres Freund <andres@2ndquadrant.com>
Re: Add more regression tests for dbcommands Robins Tharakan <tharakan@gmail.com>
Re: Add more regression tests for dbcommands Andres Freund <andres@2ndquadrant.com>
Re: Add more regression tests for dbcommands Tom Lane <tgl@sss.pgh.pa.us>
Re: Add more regression tests for dbcommands Peter Eisentraut <peter_e@gmx.net>
Re: Add more regression tests for dbcommands Tom Lane <tgl@sss.pgh.pa.us>
Re: Add more regression tests for dbcommands Peter Eisentraut <peter_e@gmx.net>
Re: Add more regression tests for dbcommands Fabien COELHO <coelho@cri.ensmp.fr>
Re: Add more regression tests for dbcommands Robins Tharakan <tharakan@gmail.com>
Re: Add more regression tests for dbcommands Robins Tharakan <tharakan@gmail.com>
Re: Add more regression tests for dbcommands Robert Haas <robertmhaas@gmail.com>
Hi Andres,
Just an aside, your point about CONNECTION LIMIT was something that just didn't come to my mind and is probably a good way to test ALTER DATABASE with CONNECTION LIMIT.
Its just that that actually wasn't what I was testing there. That 'CONNECTION LIMIT' test was coupled with CREATE DATABASE because I wanted to test that 'branch' in the CREATE DATABASE function just to ensure that there was a regression test that tests CONNECTION LIMIT specifically with CREATE DATABASE. That's all. A check to confirm whether connection limit restrictions actually got enforced was something I missed, but well, its out of the window for now anyway.
Its just that that actually wasn't what I was testing there. That 'CONNECTION LIMIT' test was coupled with CREATE DATABASE because I wanted to test that 'branch' in the CREATE DATABASE function just to ensure that there was a regression test that tests CONNECTION LIMIT specifically with CREATE DATABASE. That's all. A check to confirm whether connection limit restrictions actually got enforced was something I missed, but well, its out of the window for now anyway.
--
Robins Tharakan
On 26 June 2013 06:34, Andres Freund <andres@2ndquadrant.com> wrote:
Hi,
I am generally a bit unsure whether the regression tests you propose
aren't a bit too verbose. Does any of the committers have an opinion
about this?
My feeling is that they are ok if they aren't slowing down things much.You should be able to test that by setting the connection limit to 1 and
On 2013-06-26 01:55:53 -0500, Robins Tharakan wrote:
> The CREATE DATABASE test itself was checking whether the 'CONNECTION LIMIT'
> was working. Removed that as well.
then try to connect using \c. The old connection is only dropped after
the new one has been successfully performed.
Greetings,
Andres Freund
--
Andres Freund http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
В списке pgsql-hackers по дате отправления
