Re: pgsql: Rework error messages around file handling

2018-07-17 Thread Michael Paquier
On Tue, Jul 17, 2018 at 09:13:28PM -0400, Tom Lane wrote: > Hm, I'd just tried this on my Mac laptop and saw only the one in > walsender.c. No objection to the other two though, they might be > relevant on other platforms. longfin has turned back to green again, so at least this is fixed now. --

Re: pgsql: Rework error messages around file handling

2018-07-17 Thread Tom Lane
Michael Paquier writes: > This attempt made longfin happier for the path patched, but there were a > couple of other ones which were hidden behind. I got them fixed now, > which should bring the machine back to green. Hm, I'd just tried this on my Mac laptop and saw only the one in walsender.c.

Re: pgsql: Rework error messages around file handling

2018-07-17 Thread Michael Paquier
On Wed, Jul 18, 2018 at 09:14:14AM +0900, Michael Paquier wrote: > Thanks for double-checking, pushed. This attempt made longfin happier for the path patched, but there were a couple of other ones which were hidden behind. I got them fixed now, which should bring the machine back to green. -- Mic

Re: pgsql: Rework error messages around file handling

2018-07-17 Thread Michael Paquier
On Tue, Jul 17, 2018 at 08:02:43PM -0400, Tom Lane wrote: > Sounds right to me. Thanks for double-checking, pushed. -- Michael signature.asc Description: PGP signature

Re: pgsql: Rework error messages around file handling

2018-07-17 Thread Tom Lane
Michael Paquier writes: > On Tue, Jul 17, 2018 at 07:23:04PM -0400, Tom Lane wrote: >> longfin thinks you missed at least one cast ... > Hm, I checked that. Just casting to (Size) should do it. Sounds right to me. regards, tom lane

Re: pgsql: Rework error messages around file handling

2018-07-17 Thread Michael Paquier
On Tue, Jul 17, 2018 at 07:23:04PM -0400, Tom Lane wrote: > Michael Paquier writes: > > Rework error messages around file handling > > longfin thinks you missed at least one cast ... Hm, I checked that. Just casting to (Size) should do it. I cannot get a warning with neither clang nor gcc on D

Re: pgsql: Rework error messages around file handling

2018-07-17 Thread Tom Lane
Michael Paquier writes: > Rework error messages around file handling longfin thinks you missed at least one cast ... regards, tom lane

pgsql: Rework error messages around file handling

2018-07-17 Thread Michael Paquier
Rework error messages around file handling Some error messages related to file handling are using the code path context to define their state. For example, 2PC-related errors are referring to "two-phase status files", or "relation mapping file" is used for catalog-to-filenode mapping, however tho