Re: Dead code with short varlenas in toast_save_datum() - Mailing list pgsql-hackers

From Nikita Malakhov
Subject Re: Dead code with short varlenas in toast_save_datum()
Date
Msg-id CAN-LCVM5+J_QvTX7FwywCJFXAba3y8XUCbM+O+gY7qyesxWjjQ@mail.gmail.com
Whole thread Raw
In response to Re: Dead code with short varlenas in toast_save_datum()  (Nikhil Kumar Veldanda <veldanda.nikhilkumar17@gmail.com>)
List pgsql-hackers
Hi,

Nikhil, thank you! This case should be added to regressions.

On Tue, Aug 5, 2025 at 8:26 PM Nikhil Kumar Veldanda <veldanda.nikhilkumar17@gmail.com> wrote:
Hi all,

On Sun, Aug 3, 2025 at 8:16 PM Michael Paquier <michael@paquier.xyz> wrote:
>     if (VARATT_IS_SHORT(dval))
>     {
>         data_p = VARDATA_SHORT(dval);
>         data_todo = VARSIZE_SHORT(dval) - VARHDRSZ_SHORT;
>         toast_pointer.va_rawsize = data_todo + VARHDRSZ;    /* as if not short */
>         toast_pointer.va_extinfo = data_todo;
>     }
>
> Coverage link:
> https://coverage.postgresql.org/src/backend/access/common/toast_internals.c.gcov.html
>

This code path is currently not covered by tests. It can be exercised
with the following SQL pattern

CREATE TABLE temp_tbl (a text, b text);
ALTER TABLE temp_tbl SET (toast_tuple_target = 128);
ALTER TABLE temp_tbl ALTER COLUMN a SET STORAGE EXTERNAL;
ALTER TABLE temp_tbl ALTER COLUMN b SET STORAGE EXTERNAL;
INSERT INTO temp_tbl values(repeat('a', 4000), repeat('a', 120));

--
Nikhil Veldanda




--
Regards,
Nikita Malakhov
Postgres Professional
The Russian Postgres Company

pgsql-hackers by date:

Previous
From: Nathan Bossart
Date:
Subject: Re: More protocol.h replacements this time into walsender.c
Next
From: Tomas Vondra
Date:
Subject: Re: Bug in brin_minmax_multi_distance_numeric()