Change LogicalTapeSetBlocks() to use nBlocksWritten.
authorJeff Davis <jdavis@postgresql.org>
Wed, 16 Sep 2020 04:34:05 +0000 (21:34 -0700)
committerJeff Davis <jdavis@postgresql.org>
Wed, 16 Sep 2020 04:42:25 +0000 (21:42 -0700)
commitc8aeaf3ab31edeedf1791e37c74bcedf61a916ed
tree601b35f13b5a01780faeda155851ff80646a6af8
parent3bd35d4f516adfc492360b20e72911949c961e47
Change LogicalTapeSetBlocks() to use nBlocksWritten.

Previously, it was based on nBlocksAllocated to account for tapes with
open write buffers that may not have made it to the BufFile yet.

That was unnecessary, because callers do not need to get the number of
blocks while a tape has an open write buffer; and it also conflicted
with the preallocation logic added for HashAgg.

Reviewed-by: Peter Geoghegan
Discussion: https://postgr.es/m/ce5af05900fdbd0e9185747825a7423c48501964.camel@j-davis.com
Backpatch-through: 13
src/backend/executor/nodeAgg.c
src/backend/utils/sort/logtape.c