Re: client-side fsync() error handling

On 2020-02-21 05:18, Michael Paquier wrote:
> On Thu, Feb 20, 2020 at 10:10:11AM +0100, Peter Eisentraut wrote:
>> OK, added in new patch.
> 
> Thanks, that looks good.

committed

>> The frontends do neither right now, or at least the error handling is very
>> inconsistent and inscrutable.  It would be possible in theory to add a retry
>> option, but that would be a very different patch, and given what we have
>> learned about fsync(), it probably wouldn't be widely useful.
> 
> Perhaps.  Let's have this discussion later if there are drawbacks
> about changing things the way your patch does.  If we don't do that,
> we'll never know about it either and this patch makes things safer.

yup

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services