Re: CLUSTER sort on abbreviated expressions is broken - Mailing list pgsql-hackers

From Thomas Munro
Subject Re: CLUSTER sort on abbreviated expressions is broken
Date
Msg-id CA+hUKGK=4osGviPYciALX99jUm3Fp8moS0FxYkqxTMnMbJc-4g@mail.gmail.com
Whole thread Raw
In response to Re: CLUSTER sort on abbreviated expressions is broken  (Peter Geoghegan <pg@bowt.ie>)
Responses Re: CLUSTER sort on abbreviated expressions is broken  (Peter Geoghegan <pg@bowt.ie>)
List pgsql-hackers
On Mon, Apr 4, 2022 at 11:12 AM Peter Geoghegan <pg@bowt.ie> wrote:
> We will need a backpatchable fix, since Thomas' recent fix (commit
> cc58eecc5d75a9329a6d49a25a6499aea7ee6fd6) only targeted the master
> branch.

I probably should have made it clearer in the commit message,
cc58eecc5 doesn't fix this problem in the master branch.  It only
fixes the code that incorrectly assumed that datum1 was always
available.  Now it skips the optimised path, and falls back to the
slow path, that still has *this* bug, and the test upthread still
fails.  I wrote about this separately because it's clearly independent
and I didn't want it to be mistaken for an open item for 15.



pgsql-hackers by date:

Previous
From: Peter Geoghegan
Date:
Subject: Re: CLUSTER sort on abbreviated expressions is broken
Next
From: "David G. Johnston"
Date:
Subject: Re: unlogged sequences