From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql: Simplify plpgsql's check for simple expressions. |
Date: | 2017-08-16 16:53:53 |
Message-ID: | CA+TgmobV+=2NiD99Z08VAXik7txRRxKMLNVkjZhiJ48ZAp2rfQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On Wed, Aug 16, 2017 at 12:49 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Robert Haas <robertmhaas(at)gmail(dot)com> writes:
>> On Wed, Aug 16, 2017 at 11:56 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>>> I can get on board with that statement. Can you draft a better wording?
>
>> Here is an attempt. Feel free to edit.
>
> I think s/plan/query/ in the last bit would be better. Perhaps
>
> + * However, if force_parallel_mode = on or force_parallel_mode = regress,
> + * then we impose parallel mode whenever it's safe to do so, even if the
> + * final plan doesn't use parallelism. It's not safe to do so if the query
> + * contains anything parallel-unsafe; parallelModeOK will be false in that
> + * case. Otherwise, everything in the query is either parallel-safe or
> + * parallel-restricted, and in either case it should be OK to impose
> + * parallel-mode restrictions. If that ends up breaking something, then
> + * either some function the user included in the query is incorrectly
> + * labelled as parallel-safe or parallel-restricted when in reality it's
> + * parallel-unsafe, or else the query planner itself has a bug.
> */
Works for me. I'm happy to phrase this in any way that makes it clear
to you, 'cuz it's already clear to me. :-)
You want to push something, or should I do it?
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2017-08-16 16:57:58 | Re: [COMMITTERS] pgsql: Simplify plpgsql's check for simple expressions. |
Previous Message | Tom Lane | 2017-08-16 16:49:32 | Re: [COMMITTERS] pgsql: Simplify plpgsql's check for simple expressions. |
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2017-08-16 16:55:15 | Re: Atomics for heap_parallelscan_nextpage() |
Previous Message | Tom Lane | 2017-08-16 16:49:32 | Re: [COMMITTERS] pgsql: Simplify plpgsql's check for simple expressions. |