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
