pgsql: Fix missing abort checks in pg_backup_directory.c.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix missing abort checks in pg_backup_directory.c.
Date: 2016-05-29 17:19:04
Message-ID: E1b74Mi-0006Lz-0Y@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix missing abort checks in pg_backup_directory.c.

Parallel restore from directory format failed to respond to control-C
in a timely manner, because there were no checkAborting() calls in the
code path that reads data from a file and sends it to the backend.
If any worker was in the midst of restoring data for a large table,
you'd just have to wait.

This fix doesn't do anything for the problem of aborting a long-running
server-side command, but at least it fixes things for data transfers.

Back-patch to 9.3 where parallel restore was introduced.

Branch
------
REL9_3_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/3033e7359fb850d71ffe9deb88d013cf83a98ad2

Modified Files
--------------
src/bin/pg_dump/pg_backup_directory.c | 8 ++++++++
1 file changed, 8 insertions(+)

Browse pgsql-committers by date

  From Date Subject
Next Message Alvaro Herrera 2016-05-30 18:48:44 pgsql: Fix PageAddItem BRIN bug
Previous Message Tom Lane 2016-05-29 17:00:25 pgsql: Remove pg_dump/parallel.c's useless "aborting" flag.