From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Andres Freund <andres(at)anarazel(dot)de> |
Cc: | "Drouvot, Bertrand" <bertranddrouvot(dot)pg(at)gmail(dot)com>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Ibrar Ahmed <ibrar(dot)ahmad(at)gmail(dot)com>, Amit Khandekar <amitdkhan(dot)pg(at)gmail(dot)com>, fabriziomello(at)gmail(dot)com, tushar <tushar(dot)ahuja(at)enterprisedb(dot)com>, Rahila Syed <rahila(dot)syed(at)2ndquadrant(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Minimal logical decoding on standbys |
Date: | 2022-12-20 20:39:26 |
Message-ID: | CA+TgmoaOsoQuXEo6gMq0GP8SyvCaS=ptYqdYc0uzDMoeShf_nw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, Dec 20, 2022 at 1:25 PM Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> I'm going to go through all the record types one by one before
> commenting further.
OK, so xl_hash_vacuum_one_page, at least, is a live issue. To reproduce:
./configure <whatever your usual options are>
echo 'COPT=-fsanitize=alignment -fno-sanitize-recover=all' > src/Makefile.custom
make -j8
make install
initdb
postgres
Then in another window:
pg_basebackup -D pgstandby -R
# edit postgresql.conf, change port number
postgres -D pgstandby
Then in a third window, using the attached files:
pgbench -i -s 10
psql -f kpt_hash_setup.sql
pgbench -T 10 -c 4 -j 4 -f kpt_hash_pgbench.sql
With the patch, the standby falls over:
bufpage.c:1194:31: runtime error: load of misaligned address
0x7fa62f05d119 for type 'OffsetNumber' (aka 'unsigned short'), which
requires 2 byte alignment
0x7fa62f05d119: note: pointer points here
00 00 00 00 e5 00 8f 00 00 00 00 87 00 ab 20 20 20 20 20 20 20 20
20 20 20 20 20 20 20 20 20 20
^
Without the patch, this doesn't occur.
I think this might be the only WAL record type where there's a
problem, but I haven't fully confirmed that yet.
--
Robert Haas
EDB: http://www.enterprisedb.com
Attachment | Content-Type | Size |
---|---|---|
kpt_hash_setup.sql | application/octet-stream | 416 bytes |
kpt_hash_pgbench.sql | application/octet-stream | 615 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | David Rowley | 2022-12-20 21:05:05 | Re: appendBinaryStringInfo stuff |
Previous Message | Drouvot, Bertrand | 2022-12-20 20:35:13 | Re: Minimal logical decoding on standbys |