On Sun, Mar 12, 2017 at 3:05 PM, Peter Geoghegan <p...@bowt.ie> wrote: > I attach my V9 of the patch. I came up some stuff for the design of > resource management that I think meets every design goal that we have > for shared/unified BufFiles:
Commit 2609e91fc broke the parallel CREATE INDEX cost model. I should now pass -1 as the index block argument to compute_parallel_worker(), just as all callers that aren't parallel index scan do after that commit. This issue caused V9 to never choose parallel CREATE INDEX within nbtsort.c. There was also a small amount of bitrot. Attached V10 fixes this regression. I also couldn't resist adding a few new assertions that I thought were worth having to buffile.c, plus dedicated wait events for parallel tuplesort. And, I fixed a silly bug added in V9 around where worker_wait() should occur. -- Peter Geoghegan
0001-Add-parallel-B-tree-index-build-sorting.patch.gz
Description: GNU Zip compressed data
0002-Add-temporary-testing-tools.patch.gz
Description: GNU Zip compressed data
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers