On Wed, Jul 27, 2022 at 5:10 PM Xing Guo <higuox...@gmail.com> wrote:
> The bounded heap sorting status flag is set twice in sort_bounded_heap() > and tuplesort_performsort(). This patch helps remove one of them. > Revisiting this patch I think maybe it's better to remove the setting of Tuplesort status from tuplesort_performsort() for the TSS_BOUNDED case. Thus we keep the heap manipulation routines, make_bounded_heap and sort_bounded_heap, consistent in that they update their status accordingly inside the function. Also, would you please add it to the CF to not lose track of it? Thanks Richard
