]> granicus.if.org Git - postgresql/commit
Improve parallel restore's ability to cope with selective restore (-L option).
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 21 Aug 2010 13:59:44 +0000 (13:59 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 21 Aug 2010 13:59:44 +0000 (13:59 +0000)
commitc5d6d5bc6d23b6ebd6cb17d401e10581cd112c20
treedd1f99184ecdce7b08727e783626a451785303e2
parent5abd2d704d60fc58776ae84eaf54283a2cb2ceb2
Improve parallel restore's ability to cope with selective restore (-L option).

The original coding tended to break down in the face of modified restore
orders, as shown in bug #5626 from Albert Ullrich, because it would flip over
into parallel-restore operation too soon.  That causes problems because we
don't have sufficient dependency information in dump archives to allow safe
parallel processing of SECTION_PRE_DATA items.  Even if we did, it's probably
undesirable to allow that to override the commanded restore order.

To fix the problem of omitted items causing unexpected changes in restore
order, tweak SortTocFromFile so that omitted items end up at the head of
the list not the tail.  This ensures that they'll be examined and their
dependencies will be marked satisfied before we get to any interesting
items.

In HEAD and 9.0, we can easily change restore_toc_entries_parallel so that
all SECTION_PRE_DATA items are guaranteed to be processed in the initial
serial-restore loop, and hence in commanded order.  Only DATA and POST_DATA
items are candidates for parallel processing.  For them there might be
variations from the commanded order because of parallelism, but we should
do it in a safe order thanks to dependencies.

In 8.4 it's much harder to make such a guarantee.  I settled for not
letting the initial loop break out into parallel processing mode if
it sees a DATA/POST_DATA item that's not to be restored; this at least
prevents a non-restorable item from causing premature exit from the loop.
This means that 8.4 will be more likely to fail given a badly-ordered -L
list than 9.x, but we don't really promise any such thing will work anyway.
src/bin/pg_dump/pg_backup_archiver.c