Re: Parallel tuplesort (for parallel B-Tree index creation)
Peter Geoghegan <pg@heroku.com>
On Tue, Sep 6, 2016 at 11:09 PM, Heikki Linnakangas <hlinnaka@iki.fi> wrote: >> The big picture here is that you can't only USEMEM() for tapes as the >> need arises for new tapes as new runs are created. You'll just run a >> massive availMem deficit, that you have no way of paying back, because >> you can't "liquidate assets to pay off your creditors" (e.g., release >> a bit of the memtuples memory). The fact is that memtuples growth >> doesn't work that way. The memtuples array never shrinks. > > > Hmm. But memtuples is empty, just after we have built the initial runs. Why > couldn't we shrink, i.e. free and reallocate, it? After we've built the initial runs, we do in fact give a FREEMEM() refund to those tapes that were not used within beginmerge(), as I mentioned just now (with a high workMem, this is often the great majority of many thousands of logical tapes -- that's how you get to wasting 8% of 5GB of maintenance_work_mem). What's at issue with this 500 tapes cap patch is what happens after tuples are first dumped (after we decide that this is going to be an external sort -- where we call tuplesort_merge_order() to get the number of logical tapes in the tapeset), but before the final merge happens, where we're already doing the right thing for merging by giving that refund. I want to stop logical allocation (USEMEM()) of an enormous number of tapes, to make run generation itself able to use more memory. It's surprisingly difficult to do something cleverer than just impose a cap. -- Peter Geoghegan
Commits
-
Support parallel btree index builds.
- 9da0cc35284b 11.0 landed
-
Report an ERROR if a parallel worker fails to start properly.
- 2badb5afb89c 11.0 cited
-
Transfer state pertaining to pending REINDEX operations to workers.
- 29d58fd3adae 11.0 landed
-
Add a barrier primitive for synchronizing backends.
- 1145acc70deb 11.0 cited
-
Allow DML commands that create tables to use parallel query.
- e9baa5e9fa14 11.0 cited
-
Refactor GetOldestXmin() to use flags
- af4b1a0869bd 10.0 cited
-
Fix regression in parallel planning against inheritance tables.
- 2609e91fcf9d 10.0 cited
-
Don't create "holes" in BufFiles, in the new logtape code.
- 7ac4a389a7db 10.0 cited
-
Simplify the code for logical tape read buffers.
- b75f467b6eec 10.0 cited
-
Fix excessive memory consumption in the new sort pre-reading code.
- b56fb691b003 10.0 cited
-
Implement binary heap replace-top operation in a smarter way.
- 24598337c8d2 10.0 cited
-
Cosmetic code cleanup in commands/extension.c.
- 25794e841e5b 10.0 cited
-
Speed up planner's scanning for parallel-query hazards.
- da1c91631e35 10.0 cited
-
Read from the same worker repeatedly until it returns no tuple.
- bc7fcab5e36b 9.6.0 cited
-
Improve tuplesort.c to support variable merge order. The original coding
- df700e6b4019 8.2.0 cited