Re: BUG #17564: Planner bug in combination of generate_series(), unnest() and ORDER BY - Mailing list pgsql-bugs

From Martijn van Oosterhout
Subject Re: BUG #17564: Planner bug in combination of generate_series(), unnest() and ORDER BY
Date
Msg-id CADWG95uxFuobe8AoXdYLCTzDBWo3JUkHS7m6J=X4m6qdn4CkTQ@mail.gmail.com
Whole thread Raw
In response to Re: BUG #17564: Planner bug in combination of generate_series(), unnest() and ORDER BY  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: BUG #17564: Planner bug in combination of generate_series(), unnest() and ORDER BY
List pgsql-bugs
Hoi Tom,

On Wed, 3 Aug 2022 at 16:56, Tom Lane <tgl@sss.pgh.pa.us> wrote:
Richard Guo <guofenglinux@gmail.com> writes:
> On Tue, Aug 2, 2022 at 4:50 PM Martijn van Oosterhout <kleptog@gmail.com>
> wrote:
>> Now it's morning I've thought of a way to reproduce it more easily, see
>> the attached script.

> Thanks for the report! I can reproduce it on HEAD.

FWIW, this reproduces the bug for me in v13 and v14, but not v15 or HEAD.
While the method to fix the bug seems clear enough, it doesn't seem
like we have a test case that's stable enough to be worth anything
as a regression test.  Hmmm...


Looking at what Richard has uncovered, it appears the issue is related to the table simply being big enough that it considers a parallel seqscan plan, and then fails. Something I can look into in the morning.

The part I haven't seen explained is why the generate_series() is important. My guess is that if you replace it with an expression it is no longer an SRF and it produces some completely different plan that prevents the problematic path being triggered.

Nice that the problem has been found!
--

pgsql-bugs by date:

Previous
From: Brad Nicholson
Date:
Subject: No-op updates with partitioning and logical replication started failing in version 13
Next
From: Tom Lane
Date:
Subject: Re: BUG #17564: Planner bug in combination of generate_series(), unnest() and ORDER BY