| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Michael Paquier <michael(at)paquier(dot)xyz> |
| Cc: | pgsql-committers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: pgsql: Address set of issues with errno handling |
| Date: | 2018-08-04 15:21:28 |
| Message-ID: | 22991.1533396088@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers |
Michael Paquier <michael(at)paquier(dot)xyz> writes:
> On Fri, Aug 03, 2018 at 04:04:36PM -0400, Tom Lane wrote:
>> You need to go back and add the pre-clearing of errno in each of these
>> places, otherwise the added code is basically useless.
> I looked at all code paths enforcing ENOSPC on write() calls, and
> attached is a patch to address this issue for all of them. What do you
> think? I can of course get that addressed before the next set of minor
> releases.
That looks good as far as it goes. I didn't cross-check that you
hit everyplace that needs this, but if you grepped for references to
ENOSPC then you presumably found them all.
IMO, it's OK to push this this weekend if you have time.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Michael Paquier | 2018-08-04 20:34:40 | pgsql: Reset properly errno before calling write() |
| Previous Message | Michael Paquier | 2018-08-04 12:18:34 | Re: pgsql: Address set of issues with errno handling |