Re: checking return value from unlink in write_relcache_init_file - Mailing list pgsql-hackers

From Tom Lane
Subject Re: checking return value from unlink in write_relcache_init_file
Date
Msg-id 111890.1622769374@sss.pgh.pa.us
Whole thread Raw
In response to Re: checking return value from unlink in write_relcache_init_file  (Alvaro Herrera <alvherre@alvh.no-ip.org>)
Responses Re: checking return value from unlink in write_relcache_init_file  (Zhihong Yu <zyu@yugabyte.com>)
List pgsql-hackers
Alvaro Herrera <alvherre@alvh.no-ip.org> writes:
> On 2021-Jun-03, Tom Lane wrote:
>> If the unlink fails, there's only really a problem if the subsequent
>> open() fails to overwrite the file --- and that stanza is perfectly
>> capable of complaining for itself.  So I think the code is fine and
>> there's no need for a separate message about the unlink.  Refusing to
>> proceed, as you've done here, is strictly worse than what we have.

> It does seem to deserve a comment explaining this.

Agreed, the existing comment there is a tad terse.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: BUG #16079: Question Regarding the BUG #16064
Next
From: Zhihong Yu
Date:
Subject: Re: checking return value from unlink in write_relcache_init_file