Minor cleanup of the BRIN parallel build code
authorTomas Vondra <tomas.vondra@postgresql.org>
Sat, 30 Dec 2023 21:50:54 +0000 (22:50 +0100)
committerTomas Vondra <tomas.vondra@postgresql.org>
Sat, 30 Dec 2023 22:15:04 +0000 (23:15 +0100)
commit6c63bcbf3ccf555f02a8fd8b2acaf01a6e94cb97
treec7bfb309c0aa09e27ba008b3c01e8bea1ef9c249
parent5632d6e18a8afd586a99f680eddddc38b4ead769
Minor cleanup of the BRIN parallel build code

Commit b437571714 added support for parallel builds for BRIN indexes,
using code similar to BTREE parallel builds, and also a new tuplesort
variant. This commit simplifies the new code in two ways:

* The "spool" grouping tuplesort and the heap/index is not necessary.
  The heap/index are available as separate arguments, causing confusion.
  So remove the spool, and use the tuplesort directly.

* The new tuplesort variant does not need the heap/index, as it sorts
  simply by the range block number, without accessing the tuple data.
  So simplify that too.

Initial report and patch by Ranier Vilela, further cleanup by me.

Author: Ranier Vilela
Discussion: https://postgr.es/m/CAEudQAqD7f2i4iyEaAz-5o-bf6zXVX-AkNUBm-YjUXEemaEh6A%40mail.gmail.com
src/backend/access/brin/brin.c
src/backend/utils/sort/tuplesortvariants.c
src/include/utils/tuplesort.h
src/tools/pgindent/typedefs.list