Introduce LogicalTapeSetExtend().
authorJeff Davis <[email protected]>
Sat, 7 Mar 2020 01:28:53 +0000 (17:28 -0800)
committerJeff Davis <[email protected]>
Mon, 9 Mar 2020 17:40:02 +0000 (10:40 -0700)
commit24d85952a57b16090ca8ad9cf800fbdd9ddd104f
tree7a4d0244ac49eea79d221c6287f097e64e6dec5c
parent17d3fcdc39577b1e43624392c25e5384da42e522
Introduce LogicalTapeSetExtend().

Increases the number of tapes in a logical tape set. This will be
important for disk-based hash aggregation, because the maximum number
of tapes is not known ahead of time.

While discussing this change, it was observed to regress the
performance of Sort for at least one test case. The performance
regression was because some versions of GCC switch to an inlined
version of memcpy() in LogicalTapeWrite() after this change. No
performance regression for clang was observed.

Because the regression is due to an arbitrary decision by the
compiler, I decided it shouldn't hold up this change. If it needs to
be fixed, we can find a workaround.

Author: Adam Lee, Jeff Davis
Discussion: https://postgr.es/m/e54bfec11c59689890f277722aaaabd05f78e22c.camel%40j-davis.com
src/backend/utils/sort/logtape.c
src/include/utils/logtape.h