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:47:47 +0000 (21:47 -0700)
commit6e146a663536f86c8421ac6ed08c4eb9a69979fd
tree051d985498a2599d4045ab51f4cff8a65f7aecba
parent42a46f5a76ecb435ac3a29fc3a0d03f1cfff17ab
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