On Fri, 26 Sep 2025 00:03:06 +0900 Fujii Masao <[email protected]> wrote:
> On Thu, Sep 25, 2025 at 4:22 PM Yugo Nagata <[email protected]> wrote: > > I've attached updated patches. > > Thanks for updating the patches! > > About 0001: you mentioned that the lost error message issue occurs in > pipeline mode. > Just to confirm, are you sure it never happens in non-pipeline mode? > From a quick look, > readCommandResponse() seems to have this problem regardless of whether > pipeline > mode is used. > > If it can also happen outside pipeline mode, maybe we should split this from > the assertion failure fix, since they'd need to be backpatched to > different branches. I could not find a code path that resets the error state before reporting in non-pipeline mode, since it is typically reset when starting to send a query. However, referencing an error message after another PQgetResult() does not seem like a good idea in general, so I agree with splitting the patch. I'll submit updated patches soon. Regards, Yugo Nagata -- Yugo Nagata <[email protected]>
