]> granicus.if.org Git - postgresql/commit
Eliminate some more O(N^2) behaviors in pg_dump/pg_restore.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 29 May 2012 00:38:28 +0000 (20:38 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 29 May 2012 00:38:28 +0000 (20:38 -0400)
commitc89bdf769065981e6948c94da8c0b96737bb8462
treec8ee47cf04a14eeec2d7e7d16a600bc6387ac16a
parent2d612abd4d330cd9d3f5a3f5936b20a6e837a69c
Eliminate some more O(N^2) behaviors in pg_dump/pg_restore.

This patch fixes three places (which AFAICT is all of them) where runtime
was O(N^2) in the number of TOC entries, by using an index array to replace
linear searches of the TOC list.  This performance issue is a bit less bad
than those recently fixed, because it depends on the number of items dumped
not the number in the source database, so the problem can be dodged by
doing partial dumps.

The previous coding already had an instance of one of the two index arrays
needed, but it was only calculated in parallel-restore cases; now we need
it all the time.  I also chose to move the arrays into the ArchiveHandle
data structure, to make this code a bit more ready for the day that we
try to sling multiple ArchiveHandles around in pg_dump or pg_restore.

Since we still need some server-side work before pg_dump can really cope
nicely with tens of thousands of tables, there's probably little point in
back-patching.
src/bin/pg_dump/pg_backup_archiver.c
src/bin/pg_dump/pg_backup_archiver.h