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
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. |