Since sort_bounded_heap makes state changes that should be made

Tom Lane <tgl@sss.pgh.pa.us>

Commit: d2825e1c855e2ce0e9d3e6bc2c6f3d7793394a04
Author: Tom Lane <tgl@sss.pgh.pa.us>
Date: 2007-09-01T18:47:39Z
Releases: 8.3.0
Since sort_bounded_heap makes state changes that should be made
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.

Files

PathChange+/−
src/backend/utils/sort/tuplesort.c modified +2 −3