From: | Beena Emerson <memissemerson(at)gmail(dot)com> |
---|---|
To: | Ryan Murphy <ryanfmurphy(at)gmail(dot)com> |
Cc: | "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>, Peter Eisentraut <peter_e(at)gmx(dot)net> |
Subject: | Re: Re: Clarifying "server starting" messaging in pg_ctl start without --wait |
Date: | 2017-01-11 12:36:00 |
Message-ID: | CAOG9ApGSX9fMQM2mbxvJ=DG1J6OFbjy3MN6yZwJBr4VC0Yk1iA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Jan 6, 2017 at 11:54 AM, Ryan Murphy <ryanfmurphy(at)gmail(dot)com> wrote:
> The following review has been posted through the commitfest application:
> make installcheck-world: tested, failed
> Implements feature: tested, passed
> Spec compliant: tested, passed
> Documentation: tested, passed
>
> (Though I could not check "make installcheck-world" as passed because it
> failed 1 test, I think it basically SHOULD pass - see my comment below.)
>
> Patch looks good to me and does what we talked about, and Docs seem clear
> and correct.
>
> I was able to build Postgres and run pg_ctl and observe that it waited by
> default for the 'start' action, which addresses my original concern.
>
> `make` and `make install` went fine, and `make check` did as well, but
> `make installcheck-world` said (after a while):
>
> =======================
> 1 of 55 tests failed.
> =======================
>
>
>
I am sure you would get this error even without the patch.
--
Thank you,
Beena Emerson
Have a Great Day!
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2017-01-11 12:37:40 | Re: Passing query string to workers |
Previous Message | Etsuro Fujita | 2017-01-11 12:15:34 | Re: Push down more full joins in postgres_fdw |