Hello.
At Fri, 8 Nov 2019 08:23:41 +0500, Andrey Lepikhov <a.lepikhov@postgrespro.ru> wrote in
> > Can I switch the status back to "Needs review"?
> > Regards,
> >
>
> One issue is that your patch provides small information. WAL errors
> Investigation often requires information on xid, subxacts,
> delete-on-abort/commit rels; rarely - invalidation messages etc.
Basically agrred, but it can be very large in certain cases, even if
it is rare.
By the way, in the patch xact_desc_prepare seems printing
parseed.xact_time, which is not actually set by ParsePrepareRecord.
# I missed the funtion. xl_xact_prepare is used in *two* places in
# front end.
regards.
--
Kyotaro Horiguchi
NTT Open Source Software Center