diff options
| author | Tomas Vondra <tomas.vondra@postgresql.org> | 2024-04-17 18:31:43 +0200 |
|---|---|---|
| committer | Tomas Vondra <tomas.vondra@postgresql.org> | 2024-04-17 18:31:46 +0200 |
| commit | 0c2f5552d5d47c702f5077a2df131d3125d96132 (patch) | |
| tree | 4aceedef339d92de08abe4381326e676934d26a9 /src/backend/parser/parse_expr.c | |
| parent | 0bd4b0689ba1f12fbbd9919ca76a71df3e7702a2 (diff) | |
| download | postgresql-0c2f5552d5d47c702f5077a2df131d3125d96132.tar.gz postgresql-0c2f5552d5d47c702f5077a2df131d3125d96132.zip | |
Cleanup parallel BRIN index build code
Commit b43757171470 added support for parallel builds of BRIN indexes,
using code similar to BTREE. But there were to be a couple unnecessary
differences, particularly in how the leader waits for the workers, and
merges the results. So remove these, to make the code more similar.
The leader never waited on the workersdonecv condition variable, but
simply called WaitForParallelWorkersToFinish() in _brin_end_parallel()
and then merged the per-worker results. This worked correctly, but it
seems better to do the wait and merge before _brin_end_parallel().
This commit moves the relevant code to _brin_parallel_heapscan/merge(),
which means _brin_end_parallel() remains responsible only for exiting
the parallel mode and accumulating WAL usage data.
Discussion: https://postgr.es/m/3733d042-71e1-6ae6-5fac-00c12db62db6@enterprisedb.com
Diffstat (limited to 'src/backend/parser/parse_expr.c')
0 files changed, 0 insertions, 0 deletions