• Tom Lane's avatar
    Since sort_bounded_heap makes state changes that should be made · d2825e1c
    Tom Lane authored
    regardless of the number of tuples involved, it's incorrect to skip it
    when memtupcount = 1; the number of cycles saved is minuscule anyway.
    An alternative solution would be to pull the state changes out to the
    call site in tuplesort_performsort, but keeping them near the corresponding
    changes in make_bounded_heap seems marginally cleaner.  Noticed by
    Greg Stark.
    d2825e1c
tuplesort.c 89.1 KB