farrokhi/pg_repack
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Merge pull request #87 from postgrespro/vacuum_fix

Browse Source 
Don't wait for concurrent vacuums
This commit is contained in:
Josh Kupershmidt 2016-08-05 13:12:06 -07:00 committed by GitHub
commit 8db0f58daf
1 changed files with 7 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
bin/pg_repack.c
View File

@ -66,6 +66,7 @@ const char *PROGRAM_VERSION = "unknown";
* https://github.com/reorg/pg_repack/issues/1
* d. open transactions/locks existing on other databases than the actual
* processing relation (except for locks on shared objects)
* e. VACUUMs which are always executed outside transaction blocks.
*
* Note, there is some redundancy in how the filtering is done (e.g. excluding
* based on pg_backend_pid() and application_name), but that shouldn't hurt
@ -84,6 +85,8 @@ const char *PROGRAM_VERSION = "unknown";
" AND l.pid NOT IN (pg_backend_pid(), $1) " \
" AND (l.virtualxid, l.virtualtransaction) <> ('1/1', '-1/0') " \
" AND (a.application_name IS NULL OR a.application_name <> $2)" \
" AND a.query !~* E'^\\\\s*vacuum\\\\s+' " \
" AND a.query !~ E'^autovacuum: ' " \
" AND ((d.datname IS NULL OR d.datname = current_database()) OR l.database = 0)"
#define SQL_XID_SNAPSHOT_90000 \
@ -97,6 +100,8 @@ const char *PROGRAM_VERSION = "unknown";
" AND l.pid NOT IN (pg_backend_pid(), $1) " \
" AND (l.virtualxid, l.virtualtransaction) <> ('1/1', '-1/0') " \
" AND (a.application_name IS NULL OR a.application_name <> $2)" \
" AND a.query !~* E'^\\\\s*vacuum\\\\s+' " \
" AND a.query !~ E'^autovacuum: ' " \
" AND ((d.datname IS NULL OR d.datname = current_database()) OR l.database = 0)"
/* application_name is not available before 9.0. The last clause of
@ -111,6 +116,8 @@ const char *PROGRAM_VERSION = "unknown";
" ON a.datid = d.oid " \
" WHERE l.locktype = 'virtualxid' AND l.pid NOT IN (pg_backend_pid(), $1)" \
" AND (l.virtualxid, l.virtualtransaction) <> ('1/1', '-1/0') " \
" AND a.query !~* E'^\\\\s*vacuum\\\\s+' " \
" AND a.query !~ E'^autovacuum: ' " \
" AND ((d.datname IS NULL OR d.datname = current_database()) OR l.database = 0)" \
" AND ($2::text IS NOT NULL)"