pgsql: Fix missing abort checks in pg_backup_directory.c. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Fix missing abort checks in pg_backup_directory.c.
Date
Msg-id E1b74Mi-0006Lu-0H@gemulon.postgresql.org
Whole thread Raw
List 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
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/3c8aa6654a44837a2c60fc6061665df1adfd677c

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


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Remove pg_dump/parallel.c's useless "aborting" flag.
Next
From: Tom Lane
Date:
Subject: pgsql: Fix missing abort checks in pg_backup_directory.c.