pg_logical_emit_message() misses a XLogFlush() - Mailing list pgsql-hackers

From Michael Paquier
Subject pg_logical_emit_message() misses a XLogFlush()
Date
Msg-id ZNsdThSe2qgsfs7R@paquier.xyz
Whole thread Raw
Responses Re: pg_logical_emit_message() misses a XLogFlush()
List pgsql-hackers
Hi all,

While playing with pg_logical_emit_message() and WAL replay, I have
noticed that LogLogicalMessage() inserts a record but forgets to make
sure that the record has been flushed.  So, for example, if the system
crashes the message inserted can get lost.

I was writing some TAP tests for it for the sake of a bug, and I have
found this the current behavior annoying because one cannot really
rely on it when emulating crashes.

This has been introduced in 3fe3511 (from 2016), and there is no
mention of that on the original thread that led to this commit:
https://www.postgresql.org/message-id/flat/5685F999.6010202%402ndquadrant.com

This could be an issue for anybody using LogLogicalMessage() out of
core, as well, because it would mean some records lost.  So, perhaps
this should be treated as a bug, sufficient for a backpatch?

Thoughts?
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Pavel Stehule
Date:
Subject: Re: proposal: jsonb_populate_array
Next
From: Michael Paquier
Date:
Subject: Re: A Question about InvokeObjectPostAlterHook