]> granicus.if.org Git - postgresql/commit
Simplify do_pg_start_backup's API by opening pg_tblspc internally.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 4 Dec 2017 23:37:54 +0000 (18:37 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 4 Dec 2017 23:37:54 +0000 (18:37 -0500)
commit066bc21c0e085e2642ff25cc665c4efad3669d6f
tree027c786114bb35dad2ad7bb4af9aea5736963824
parent561885db05d3296082ce8750805b8ec322cf9aa1
Simplify do_pg_start_backup's API by opening pg_tblspc internally.

do_pg_start_backup() expects its callers to pass in an open DIR pointer
for the pg_tblspc directory, but there's no apparent advantage in that.
It complicates the callers without adding any flexibility, and there's no
robustness advantage, since we surely have to be prepared for errors during
the scan of pg_tblspc anyway.  In fact, by holding an extra kernel resource
during operations like the preliminary checkpoint, we might be making
things a fraction more failure-prone not less.  Hence, remove that argument
and open the directory just for the duration of the actual scan.

Discussion: https://postgr.es/m/28752.1512413887@sss.pgh.pa.us
src/backend/access/transam/xlog.c
src/backend/access/transam/xlogfuncs.c
src/backend/replication/basebackup.c
src/include/access/xlog.h