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  <br...@momjian.us>        https://momjian.us
  EDB                                      https://enterprisedb.com

  If only the physical world exists, free will is an illusion.



Reply via email to