From: Tom Lane <tgl@sss.pgh.pa.us>
To: David Rowley <dgrowleyml@gmail.com>
cc: cyg0810@gmail.com,
        PostgreSQL mailing lists <pgsql-bugs@lists.postgresql.org>
Subject: Re: BUG #17117: FailedAssertion at planner.c
In-reply-to: 
 <CAApHDvqy1GPi3OYXVXKH3hLpiNp9FdRHjQj=3cp-vhWt94Y0Nw@mail.gmail.com>
References: <17117-e7ed6c3108953c08@postgresql.org>
 <CAApHDvorY9UMz1N4B81Qu7nKc8hD0XvE3Fm0ux-dx+81F_rgxg@mail.gmail.com>
 <CAApHDvqy1GPi3OYXVXKH3hLpiNp9FdRHjQj=3cp-vhWt94Y0Nw@mail.gmail.com>
Comments: In-reply-to David Rowley <dgrowleyml@gmail.com>
	message dated "Fri, 23 Jul 2021 01:58:50 +1200"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <781664.1626965418.1@sss.pgh.pa.us>
Content-Transfer-Encoding: quoted-printable
Date: Thu, 22 Jul 2021 10:50:18 -0400
Message-ID: <781665.1626965418@sss.pgh.pa.us>
Archived-At: 
 <https://www.postgresql.org/message-id/781665.1626965418%40sss.pgh.pa.us>
Precedence: bulk

David Rowley <dgrowleyml@gmail.com> writes:
>  On Fri, 23 Jul 2021 at 00:28, David Rowley <dgrowleyml@gmail.com> wrote=
:
>> I'm unsure if parse->hasModifyingCTE is not being set properly and
>> that's causing PlannerGlobal.parallelModeOK to be set incorrectly in
>> standard_planner or if we should just be never setting anything to
>> parallelModeOK that's not parse->querySource =3D=3D QSRC_ORIGINAL.

> I think the hasModifyingCTE going missing is the problem here.

I think this is a dup of the ongoing discussion at

https://www.postgresql.org/message-id/flat/CAJcOf-fAdj=3DnDKMsRhQzndm-O13N=
Y4dL6xGcEvdX5Xvbbi0V7g@mail.gmail.com

The thing that's stalled that discussion is disagreement over whether
it's necessary to make a bigger change than what you have here.

			regards, tom lane