Commit 95b52351 authored by Robert Haas's avatar Robert Haas

Remove obsolete comment.

Commit 8b304b8b removed replacement
selection, but left behind this comment text.  The optimization to
which the comment refers is not relevant without replacement
selection, because if we had so few tuples as to require only one
tape, we would have just completed the sort in memory.

Peter Geoghegan

Discussion: http://postgr.es/m/CAH2-WznqupLA8CMjp+vqzoe0yXu0DYYbQSNZxmgN76tLnAOZ_w@mail.gmail.com
parent d329dc2e
...@@ -2459,11 +2459,6 @@ mergeruns(Tuplesortstate *state) ...@@ -2459,11 +2459,6 @@ mergeruns(Tuplesortstate *state)
* Use all the remaining memory we have available for read buffers among * Use all the remaining memory we have available for read buffers among
* the input tapes. * the input tapes.
* *
* We do this only after checking for the case that we produced only one
* initial run, because there is no need to use a large read buffer when
* we're reading from a single tape. With one tape, the I/O pattern will
* be the same regardless of the buffer size.
*
* We don't try to "rebalance" the memory among tapes, when we start a new * We don't try to "rebalance" the memory among tapes, when we start a new
* merge phase, even if some tapes are inactive in the new phase. That * merge phase, even if some tapes are inactive in the new phase. That
* would be hard, because logtape.c doesn't know where one run ends and * would be hard, because logtape.c doesn't know where one run ends and
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment