pg_logical_emit_message() misses a XLogFlush()

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема pg_logical_emit_message() misses a XLogFlush()
Дата
Msg-id ZNsdThSe2qgsfs7R@paquier.xyz
обсуждение исходный текст
Ответы Re: pg_logical_emit_message() misses a XLogFlush()  (Tomas Vondra <tomas.vondra@enterprisedb.com>)
Список 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

Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Pavel Stehule
Дата:
Сообщение: Re: proposal: jsonb_populate_array
Следующее
От: Michael Paquier
Дата:
Сообщение: Re: A Question about InvokeObjectPostAlterHook