Re: Is tuplesort meant to support bounded datum sorts?

On Tue, 13 Jul 2021 at 04:10, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
> David Rowley <dgrowleyml(at)gmail(dot)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