Re: [HACKERS] [COMMITTERS] pgsql: Add max_parallel_workers GUC.

2016-12-05 Thread Robert Haas
On Sat, Dec 3, 2016 at 11:43 AM, Tom Lane wrote: > Robert Haas writes: >> On Dec 2, 2016, at 5:45 PM, Tom Lane wrote: >>> Might work. We've had very bad luck with GUC variables with >>> interdependent defaults, but maybe the user-visible knob could be a >>> percentage of max_connections or some

Re: [HACKERS] [COMMITTERS] pgsql: Add max_parallel_workers GUC.

2016-12-03 Thread Tom Lane
Robert Haas writes: > On Dec 2, 2016, at 5:45 PM, Tom Lane wrote: >> Might work. We've had very bad luck with GUC variables with >> interdependent defaults, but maybe the user-visible knob could be a >> percentage of max_connections or something like that. > Seems like overkill. Let's just redu

Re: [HACKERS] [COMMITTERS] pgsql: Add max_parallel_workers GUC.

2016-12-02 Thread Robert Haas
On Dec 2, 2016, at 5:45 PM, Tom Lane wrote: > Jim Nasby writes: >>> On 12/2/16 2:34 PM, Robert Haas wrote: >>> Signs point to "no". It seemed like a good idea to leave some daylight >>> between max_parallel_workers and max_worker_processes, but evidently this >>> wasn't the way to get there. O

Re: [HACKERS] [COMMITTERS] pgsql: Add max_parallel_workers GUC.

2016-12-02 Thread Tom Lane
Jim Nasby writes: > On 12/2/16 2:34 PM, Robert Haas wrote: >> Signs point to "no". It seemed like a good idea to leave some daylight >> between max_parallel_workers and max_worker_processes, but evidently this >> wasn't the way to get there. Or else we should just give up on that thought. > Co

Re: [HACKERS] [COMMITTERS] pgsql: Add max_parallel_workers GUC.

2016-12-02 Thread Jim Nasby
On 12/2/16 2:34 PM, Robert Haas wrote: Signs point to "no". It seemed like a good idea to leave some daylight between max_parallel_workers and max_worker_processes, but evidently this wasn't the way to get there. Or else we should just give up on that thought. Could the defaults be scaled ba

Re: [HACKERS] [COMMITTERS] pgsql: Add max_parallel_workers GUC.

2016-12-02 Thread Robert Haas
On Dec 2, 2016, at 4:07 PM, Tom Lane wrote: > Robert Haas 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 sid

Re: [HACKERS] [COMMITTERS] pgsql: Add max_parallel_workers GUC.

2016-12-02 Thread Tom Lane
Robert Haas 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