| From: | Andres Freund <andres(at)anarazel(dot)de> |
|---|---|
| To: | Peter Smith <smithpb2250(at)gmail(dot)com> |
| Cc: | Timur Magomedov <t(dot)magomedov(at)postgrespro(dot)ru>, "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Tomas Vondra <tomas(at)vondra(dot)me>, "Aya Iwata (Fujitsu)" <iwata(dot)aya(at)fujitsu(dot)com>, Japin Li <japinli(at)hotmail(dot)com>, shveta malik <shveta(dot)malik(at)gmail(dot)com> |
| Subject: | Re: [WIP]Vertical Clustered Index (columnar store extension) - take2 |
| Date: | 2025-12-18 00:25:00 |
| Message-ID: | asznbmjvzk2xmunc4havbijkhoqepd7g2gtcw5f7fsloswug2v@msxkzvmkrnod |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
On 2025-12-17 12:40:13 +1100, Peter Smith wrote:
> Here is the latest set of VCI patches.
This fails the tests under the undefined behaviour sanitizer.
Se e.g.
https://cirrus-ci.com/task/6313371143766016
There are a lot of crashes due to ubsan failures. One of them is:
[21:22:15.116] stderr:
[21:22:15.116] # Failed test 'newline in dbname: stderr matches'
[21:22:15.116] # at /tmp/cirrus-ci-build/src/bin/pg_dump/t/003_pg_dump_with_server.pl line 29.
[21:22:15.116] # '==58220==Using libbacktrace symbolizer.
[21:22:15.116] # ../src/bin/pg_dump/pg_dump.c:20198:13: runtime error: load of value 116, which is not a valid value for type '_Bool'
[21:22:15.116] # #0 0x55ae2a204568 in addBoundaryDependencies ../src/bin/pg_dump/pg_dump.c:20198
[21:22:15.116] # #1 0x55ae2a23d768 in main ../src/bin/pg_dump/pg_dump.c:1209
[21:22:15.116] # #2 0x7f5add233ca7 (/lib/x86_64-linux-gnu/libc.so.6+0x29ca7) (BuildId: def5460e3cee00bfee25b429c97bcc4853e5b3a8)
[21:22:15.116] # #3 0x7f5add233d64 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x29d64) (BuildId: def5460e3cee00bfee25b429c97bcc4853e5b3a8)
[21:22:15.116] # #4 0x55ae2a1fbda0 in _start (/tmp/cirrus-ci-build/build/tmp_install/usr/local/pgsql/bin/pg_dump+0x55da0) (BuildId: 19f1772fd2938fdb30967788098e73d390996698)
[21:22:15.116] #
I.e. pg_dump is triggering the undefined behaviour sanitizer because of a
bogus value in a field that's supposed to be a boolean.
Greetings,
Andres
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Melanie Plageman | 2025-12-18 00:30:01 | Re: eliminate xl_heap_visible to reduce WAL (and eventually set VM on-access) |
| Previous Message | Chao Li | 2025-12-18 00:22:13 | Re: Add sanity check for duplicate enum values in GUC definitions |