| From: | Michael Paquier <michael(at)paquier(dot)xyz> |
|---|---|
| To: | Julien Rouhaud <rjuju123(at)gmail(dot)com> |
| Cc: | Fujii Masao <masao(dot)fujii(at)gmail(dot)com>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: pg_waldump and PREPARE |
| Date: | 2019-07-04 07:45:12 |
| Message-ID: | 20190704074512.GN3084@paquier.xyz |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, Jul 03, 2019 at 08:23:44PM +0200, Julien Rouhaud wrote:
> So the patch compiles and works as intended. I don't have much to say
> about it, it all looks good to me, since the concerns about xactdesc.c
> aren't worth the trouble.
>
> I'm not sure that I understand Michael's objection though, as
> xl_xact_prepare is not a new definition and AFAICS it couldn't contain
> the records anyway. So I'll let him say if he has further objections
> or if it's ready for committer!
This patch provides parsing information only for the header of the 2PC
record. Wouldn't it be interesting to get more information from the
various TwoPhaseRecordOnDisk's callbacks? We could also print much
more information in xact_desc_prepare(). Like the subxacts, the XID,
the invalidation messages and the delete-on-abort/commit rels.
--
Michael
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Michael Paquier | 2019-07-04 08:01:23 | Re: extension patch of CREATE OR REPLACE TRIGGER |
| Previous Message | Michael Paquier | 2019-07-04 07:14:09 | Re: Refactoring base64 encoding and decoding into a safer interface |