Re: pg_upgrade should truncate/remove its logs before running
Bruce Momjian <bruce@momjian.us>
From: Bruce Momjian <bruce@momjian.us>
To: Michael Paquier <michael@paquier.xyz>
Cc: Peter Eisentraut <peter.eisentraut@enterprisedb.com>, Justin Pryzby <pryzby@telsasoft.com>, Tom Lane <tgl@sss.pgh.pa.us>, Daniel Gustafsson <daniel@yesql.se>, Andrew Dunstan <andrew@dunslane.net>, pgsql-hackers@lists.postgresql.org
Date: 2022-01-24T17:39:30Z
Lists: pgsql-hackers
On Mon, Jan 24, 2022 at 10:59:40AM +0900, Michael Paquier wrote: > On Thu, Jan 20, 2022 at 07:51:37PM +0900, Michael Paquier wrote: > > Neat idea. That would work fine for my case. So I am fine to stick > > with this suggestion. > > I have been looking at this idea, and the result is quite nice, being > simpler than anything that has been proposed on this thread yet. We > get a simpler removal logic, and there is no need to perform any kind > of sanity checks with the output path provided as long as we generate > the paths and the dirs after adjust_data_dir(). ... > > <para> > <application>pg_upgrade</application> creates various working files, such > - as schema dumps, in the current working directory. For security, be sure > - that that directory is not readable or writable by any other users. > + as schema dumps, stored within <literal>pg_upgrade_output.d</literal> in > + the directory of the new cluster. > </para> Uh, how are we instructing people to delete that pg_upgrade output directory? If pg_upgrade completes cleanly, would it be removed automatically? -- Bruce Momjian <bruce@momjian.us> https://momjian.us EDB https://enterprisedb.com If only the physical world exists, free will is an illusion.
Commits
-
pg_upgrade: Move all the files generated internally to a subdirectory
- 38bfae365266 15.0 landed