On Tue, 13 Jul 2021 at 04:10, Tom Lane <t...@sss.pgh.pa.us> wrote: > > David Rowley <dgrowle...@gmail.com> writes: > > It looks like this has likely never come up before because the only > > time we use tuplesort_set_bound() is in nodeSort.c and > > nodeIncrementalSort.c, none of those currently use datum sorts. > > However, I'm thinking this is still a bug that should be fixed > > separately from Ronan's main patch. > > Yeah, I think you're right. The comment seems a little confused > though. Maybe there's no need for it at all --- there's equivalent > code in e.g. writetup_datum that has no comment.
Thanks for looking at this. I've pushed a fix and backpatched. David
