git.postgresql.org Git - postgresql.git/commitdiff

git projects / postgresql.git / commitdiff

Do not emit WAL for unlogged BRIN indexes

author Heikki Linnakangas <heikki.linnakangas@iki.fi>

2025年12月18日 13:08:48 +0000 (15:08 +0200)

committer Heikki Linnakangas <heikki.linnakangas@iki.fi>

2025年12月18日 13:08:48 +0000 (15:08 +0200)

Operations on unlogged relations should not be WAL-logged. The
brin_initialize_empty_new_buffer() function didn't get the memo.

The function is only called when a concurrent update to a brin page
uses up space that we're just about to insert to, which makes it
pretty hard to hit. If you do manage to hit it, a full-page WAL record
is erroneously emitted for the unlogged index. If you then crash,
crash recovery will fail on that record with an error like this:

FATAL: could not create file "base/5/32819": File exists

Author: Kirill Reshke <reshkekirill@gmail.com>
Discussion: https://www.postgresql.org/message-id/CALdSSPhpZXVFnWjwEBNcySx_vXtXHwB2g99gE6rK0uRJm-3GgQ@mail.gmail.com
Backpatch-through: 14

src/backend/access/brin/brin_pageops.c patch | blob | blame | history

diff --git a/src/backend/access/brin/brin_pageops.c b/src/backend/access/brin/brin_pageops.c

index 91a5ba163f2c6c0a59b2dcce7d9de6bac1ced1d0..c80b87da3d2486a8619bfded68ae5fd7418860a8 100644 (file)

--- a/src/backend/access/brin/brin_pageops.c

+++ b/src/backend/access/brin/brin_pageops.c

@@ -891,7 +891,11 @@ brin_initialize_empty_new_buffer(Relation idxrel, Buffer buffer)

page = BufferGetPage(buffer);

brin_page_init(page, BRIN_PAGETYPE_REGULAR);

MarkBufferDirty(buffer);

- log_newpage_buffer(buffer, true);

+ /* XLOG stuff */

+ if (RelationNeedsWAL(idxrel))

+ log_newpage_buffer(buffer, true);

END_CRIT_SECTION();

This is the main PostgreSQL git repository.

RSS Atom