From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Robert Haas <rhaas(at)postgresql(dot)org> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: [COMMITTERS] pgsql: Add max_parallel_workers GUC. |
Date: | 2016-12-02 21:07:46 |
Message-ID: | 20338.1480712866@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Robert Haas <rhaas(at)postgresql(dot)org> writes:
> Add max_parallel_workers GUC.
> Increase the default value of the existing max_worker_processes GUC
> from 8 to 16, and add a new max_parallel_workers GUC with a maximum
> of 8.
This broke buildfarm members coypu and sidewinder. It appears the reason
is that those machines can only get up to 30 server processes, cf this
pre-failure initdb trace:
creating directory data-C ... ok
creating subdirectories ... ok
selecting default max_connections ... 30
selecting default shared_buffers ... 128MB
selecting dynamic shared memory implementation ... sysv
creating configuration files ... ok
running bootstrap script ... ok
performing post-bootstrap initialization ... ok
syncing data to disk ... ok
So you've reduced their available number of regular backends to less than
20, which is why their tests are now dotted with
! psql: FATAL: sorry, too many clients already
There may well be other machines with similar issues; we won't know until
today's other breakage clears.
We could ask the owners of these machines to reduce the test parallelism
via the MAX_CONNECTIONS makefile variable, but I wonder whether this
increase was well thought out in the first place.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2016-12-02 22:23:57 | pgsql: Fix broken wait-for-previous-process-to-exit loop in regression |
Previous Message | Robert Haas | 2016-12-02 20:08:40 | pgsql: Fix thinko in b3427dade14cc31eb48740bc9ea98b5954470b24. |
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2016-12-02 21:09:32 | Re: Performance improvement for joins where outer side is unique |
Previous Message | Tom Lane | 2016-12-02 20:52:57 | Re: HaveNFreeProcs() iterates through entire freeProcs list |