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

From: Peter Smith <smithpb2250(at)gmail(dot)com>
To: Andres Freund <andres(at)anarazel(dot)de>
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 01:15:07
Message-ID: CAHut+PuYmwFRGPoDt4vCA35=KhSiQ5i=3Fd9VHKBtbF91SeUfQ@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Dec 18, 2025 at 11:25 AM Andres Freund <andres(at)anarazel(dot)de> wrote:
>
> 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,
>

Thanks for reporting. There has been some kind of error that has been
happening only on this platform ever since the first post of this
patch. Discovering the reason for this is on my TODO list.

Previously, the error for this platform used to manifest differently
-- reporting about a bad structure member alignment -- so I am
guessing it is the same issue, but in this case, the bad alignment
caused some bool to contain garbage.

======
Kind Regards,
Peter Smith.
Fujitsu Australia

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Euler Taveira 2025-12-18 01:28:54 Re: [Proposal] Adding Log File Capability to pg_createsubscriber
Previous Message Amit Langote 2025-12-18 01:10:18 Re: Segmentation fault on proc exit after dshash_find_or_insert