Re: pgsql: Add tests for '-f' option in dropdb utility.

From: Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
To: Amit Kapila <akapila(at)postgresql(dot)org>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
Cc: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Add tests for '-f' option in dropdb utility.
Date: 2019-11-28 13:49:48
Message-ID: CAA4eK1LC1mapqzG9N=cFoEyEPMmaUb72f+-EOeVuK4jnEFRyyw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On Thu, Nov 28, 2019 at 6:59 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
>
> On Thu, Nov 28, 2019 at 11:49 AM Amit Kapila <akapila(at)postgresql(dot)org> wrote:
> >
> > Add tests for '-f' option in dropdb utility.
> >
>
> drongo doesn't seem to like this commit [1]. Below is shown in failure report:
>
> # aborting wait: program died
> # stream contents: >>psql:<stdin>:4: server closed the connection unexpectedly
> # This probably means the server terminated abnormally
> # before or while processing the request.
> # psql:<stdin>:4: fatal: connection to server was lost
> # <<
> # pattern searched for: (?^m:FATAL: terminating connection due to
> administrator command)
>

I checked that one other similar usage in 013_crash_restart.pl is
expecting below message:
/WARNING: terminating connection because of crash of another server
process|server closed the connection unexpectedly|connection to server
was lost

The message expected by this test was: "FATAL: terminating connection
due to administrator command" which seems to be fine in most cases but
not all.

I think we just need to change the expected message the same as in
013_crash_restart.pl to make this test pass on all platforms. I will
do a bit more study and prepare a test setup in a similar environment
to test this case tomorrow. I don't want to do this today in a hurry
without testing it carefully. I hope that is fine.

--
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2019-11-28 14:46:09 Re: pgsql: Add tests for '-f' option in dropdb utility.
Previous Message Amit Kapila 2019-11-28 13:29:48 Re: pgsql: Add tests for '-f' option in dropdb utility.