Hello.

At Fri, 8 Nov 2019 08:23:41 +0500, Andrey Lepikhov <a.lepik...@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


Reply via email to