Re: [HACKERS] Re: Clarifying "server starting" messaging in pg_ctlstart without --wait
От | Beena Emerson |
---|---|
Тема | Re: [HACKERS] Re: Clarifying "server starting" messaging in pg_ctlstart without --wait |
Дата | |
Msg-id | CAOG9ApGnMFWHiRdVN_05wLKzx21TJ0C5YoUKfkbid8rCiVEorA@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: [HACKERS] Re: Clarifying "server starting" messaging in pg_ctlstart without --wait (Beena Emerson <memissemerson@gmail.com>) |
Ответы |
Re: [HACKERS] Re: Clarifying "server starting" messaging in pg_ctlstart without --wait
Re: [HACKERS] Re: Clarifying "server starting" messaging in pg_ctlstart without --wait |
Список | pgsql-hackers |
Hello,
On Wed, Jan 11, 2017 t 6:06 PM, Beena Emerson <memissemerson@gmail.com> wrote:
On Fri, Jan 6, 2017 at 11:54 AM, Ryan Murphy <ryanfmurphy@gmail.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.
The patch is good. I do not have any comments to make about the patch.
Ryan try to run 'make install-world' then 'make -i installcheck-world', -i option will ignore the error and proceed. You can check if any other tests fails. This is a separate issue, unrelated to this patch. I do not think we should stop from changing the status because of this.
The status is now updated to 'Ready for committer'
Thank you,
Beena Emerson
Have a Great Day!
В списке pgsql-hackers по дате отправления: