Re: [WIP]Vertical Clustered Index (columnar store extension) - take2

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

In response to

Responses

Browse pgsql-hackers by date

  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