Michael Paquier <mich...@paquier.xyz> writes:
> This change replaces all calls to durable_rename_excl() with
> durable_rename().  This removes the protection against accidentally
> overwriting an existing file, but some platforms are already living
> without it, and all those code paths never expect an existing file (a
> couple of assertions are added to check after that, in case).

Assert'ing that an external (filesystem) condition holds seems like a bad
idea.  See buildfarm.

                        regards, tom lane


Reply via email to