Heikki Linnakangas <heikki.linnakangas@enterprisedb.com> writes:
> This means that the WAL replay of that record type has never been tested
> correctly :-(. Looking closer at writeListPage(), why does it always
> include 'workspace' in the WAL record, even if a full-page-image is
> taken? It's not used for anything the the redo function. That's
> harmless, but bloats the WAL record unnecessary. In fact it might be
> better to never do full-page writes for that record type, since it
> completely overwrites the page anyway.
Actually, the entire thing is misdesigned from the get-go. AFAICS
it shouldn't even have its own WAL record type --- it should be using
log_newpage().
regards, tom lane
