Re: Glitches in heapam.c xlog redo routines - Mailing list pgsql-hackers

From Hiroshi Inoue
Subject Re: Glitches in heapam.c xlog redo routines
Date
Msg-id 3C2BF16B.C2784F00@tpf.co.jp
Whole thread Raw
In response to Glitches in heapam.c xlog redo routines  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Tom Lane wrote:
> 
> These things look like bugs to me:
> 
> heap_xlog_clean doesn't update the page's LSN and SUI.
> 
> heap_xlog_insert and heap_xlog_update leave random bits in the
> inserted/updated tuple's t_ctid.  heap_xlog_update leaves random
> bits in the updated tuple's t_cmax in the "move" case.

As far as I see, t_cmax isn't used anywhere once a tuple
was committed.

> 
> I am not sure about what sort of visible fault these omissions could
> produce; perhaps none.  But they look like dangerous things IMHO.
> Comments?

I'm not sure about the heap_xlog_clean case. Updating
LSN and SUI seems better to me.

regards,
Hiroshi Inoue


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: psql misbehaves because of a simple typo
Next
From: Bruce Momjian
Date:
Subject: Re: psql misbehaves because of a simple typo