public inbox for [email protected]
help / color / mirror / Atom feedFrom: Bruce Momjian <[email protected]>
To: Amit Kapila <[email protected]>
Cc: Justin Pryzby <[email protected]>
Cc: PostgreSQL Hackers <[email protected]>
Subject: Re: add default parallel query to v10 release notes? (Re: [PERFORM] performance drop after upgrade (9.6 > 10))
Date: Mon, 9 Jul 2018 11:19:28 -0400
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAA4eK1+-QkCZjiDtWQhOho5TJyD2C0odndx_u+Cwm3_p_16pLQ@mail.gmail.com>
References: <CA++JNSeCW1BSRVS7N--OmVOMDiMVA61_DFMA24fFbM2R7JdQjQ@mail.gmail.com>
<CAFj8pRDnW+bD5OmWjawxryv_=BctsE8HF5Y1n=oxgQ+xwrVqUQ@mail.gmail.com>
<CA++JNScUoxYKBhURxbG_DzoUfgKJup4a4TXUTiG1FFjuq8X-VQ@mail.gmail.com>
<CA++JNSfTEifSiBq73zpGCQuDNGr9yYcjmkx_=LtAbnadKLY_vg@mail.gmail.com>
<[email protected]>
<[email protected]>
<[email protected]>
<[email protected]>
<CAA4eK1+-QkCZjiDtWQhOho5TJyD2C0odndx_u+Cwm3_p_16pLQ@mail.gmail.com>
On Fri, Jun 22, 2018 at 02:53:36PM +0530, Amit Kapila wrote:
> On Wed, Jun 20, 2018 at 8:43 PM, Bruce Momjian <[email protected]> wrote:
> > On Thu, May 24, 2018 at 08:00:25PM -0500, Justin Pryzby wrote:
> >
> > So I did some research on this, particularly to find out how it was
> > missed in the PG 10 release notes. It turns out that
> > max_parallel_workers_per_gather has always defaulted to 2 in head, and
> > this was changed to default to 0 in the 9.6 branch:
> >
> > commit f85b1a84152f7bf019fd7a2c5eede97867dcddbb
> > Author: Robert Haas <[email protected]>
> > Date: Tue Aug 16 08:09:15 2016 -0400
> >
> > Disable parallel query by default.
> >
> > Per discussion, set the default value of max_parallel_workers_per_gather
> > to 0 in 9.6 only. We'll leave it enabled in master so that it gets
> > more testing and in the hope that it can be enable by default in v10.
> >
> > Therefore, there was no commit to find in the PG 10 commit logs. :-O
> > Not sure how we can avoid this kind of problem in the future.
> >
> > The attached patch adds a PG 10.0 release note item about this change.
> >
>
> Your proposed text looks good to me.
Done, thanks.
--
Bruce Momjian <[email protected]> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ As you are, so once was I. As I am, so you will be. +
+ Ancient Roman grave inscription +
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected]
Subject: Re: add default parallel query to v10 release notes? (Re: [PERFORM] performance drop after upgrade (9.6 > 10))
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox