From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: Zero padding in replication origin's checkpointed on disk-state. |
Date: | 2017-04-23 22:57:52 |
Message-ID: | E1d2QS0-0002Ge-8L@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Zero padding in replication origin's checkpointed on disk-state.
This seems to be largely cosmetic, avoiding valgrind bleats and the
like. The uninitialized padding influences the CRC of the on-disk
entry, but because it's also used when verifying the CRC, that doesn't
cause spurious failures. Backpatch nonetheless.
It's a bit unfortunate that contrib/test_decoding/sql/replorigin.sql
doesn't exercise the checkpoint path, but checkpoints are fairly
expensive on weaker machines, and we'd have to stop/start for that to
be meaningful.
Author: Andres Freund
Discussion: https://postgr.es/m/20170422183123.w2jgiuxtts7qrqaq@alap3.anarazel.de
Backpatch: 9.5, where replication origins were introduced
Branch
------
REL9_6_STABLE
Details
-------
https://git.postgresql.org/pg/commitdiff/39369b41451e432396aa3a913cc4c68bf0f4ee3c
Modified Files
--------------
src/backend/replication/logical/origin.c | 3 +++
1 file changed, 3 insertions(+)
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2017-04-23 22:57:53 | pgsql: Zero padding in replication origin's checkpointed on disk-state. |
Previous Message | Andres Freund | 2017-04-23 22:57:50 | pgsql: Initialize all memory for logical replication relation cache. |