Re: wrong comments in rewriteTargetListIU

From: Amit Langote <amitlangote09(at)gmail(dot)com>
To: jian he <jian(dot)universality(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: wrong comments in rewriteTargetListIU
Date: 2025-04-10 09:46:14
Message-ID: CA+HiwqGW_m1w6Np3SaGrWWAU4Ld0UAJCFSxdNOQYt94CB-WOXQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On Thu, Apr 10, 2025 at 5:58 PM jian he <jian(dot)universality(at)gmail(dot)com> wrote:
>
> hi.
>
> in function, rewriteTargetListIU
> we have:
>
> for (attrno = 1; attrno <= numattrs; attrno++)
> {
> /*
> * Can only insert DEFAULT into generated columns, regardless of
> * any OVERRIDING clauses.
> */
> if (att_tup->attgenerated && !apply_default)
> }
>
> I think the above comments are wrong, since there are no
> OVERRIDING clauses for the generated column.

Hmm, I'm not sure that is quite right. See 17958972fe3b which added
the comment, especially the test cases that it adds.

--
Thanks, Amit Langote

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Dave Cramer 2025-04-10 09:52:14 Correct documentation for protocol version
Previous Message Amit Kapila 2025-04-10 09:45:27 Re: long-standing data loss bug in initial sync of logical replication