Re: Regression in COPY FROM caused by 9f8377f7a2 - Mailing list pgsql-hackers

From Laurenz Albe
Subject Re: Regression in COPY FROM caused by 9f8377f7a2
Date
Msg-id 111178fd5653becf9e29d9637c37a2c47cf721f2.camel@cybertec.at
Whole thread Raw
In response to Regression in COPY FROM caused by 9f8377f7a2  (Laurenz Albe <laurenz.albe@cybertec.at>)
Responses Re: Regression in COPY FROM caused by 9f8377f7a2
List pgsql-hackers
On Mon, 2023-09-25 at 09:54 +0200, Laurenz Albe wrote:
> In v16 and later, the following fails:
>
> CREATE TABLE boom (t character varying(5) DEFAULT 'a long string');
>
> COPY boom FROM STDIN;
> ERROR:  value too long for type character varying(5)
>
> In PostgreSQL v15 and earlier, the COPY statement succeeds.
>
> The error is thrown in BeginCopyFrom in line 1578 (HEAD)
>
>   defexpr = expression_planner(defexpr);
>
> Bisecting shows that the regression was introduced by commit 9f8377f7a2,
> which introduced DEFAULT values for COPY FROM.

I suggest the attached fix, which evaluates default values only if
the DEFAULT option was specified or if the column does not appear in
the column list of COPY.

Yours,
Laurenz Albe

Attachment

pgsql-hackers by date:

Previous
From: "Drouvot, Bertrand"
Date:
Subject: Re: Synchronizing slots from primary to standby
Next
From: Alexander Korotkov
Date:
Subject: Re: On login trigger: take three